Add mutex for refreshing branches

We had a race condition due to refreshing branches in two different places, one which refreshed reflog commits
beforehand. The race condition meant that upon load we wouldn't see recency values (provided by the reflog commits)
against the branches

pkg/gui/controllers/helpers/refresh_helper.go

@@ -342,6 +342,9 @@ func (self *RefreshHelper) refreshStateSubmoduleConfigs() error {
 // self.refreshStatus is called at the end of this because that's when we can
 // be sure there is a State.Model.Branches array to pick the current branch from
 func (self *RefreshHelper) refreshBranches() {
+	self.c.Mutexes().RefreshingBranchesMutex.Lock()
+	defer self.c.Mutexes().RefreshingBranchesMutex.Unlock()
+
 	reflogCommits := self.c.Model().FilteredReflogCommits
 	if self.c.Modes().Filtering.Active() {
 		// in filter mode we filter our reflog commits to just those containing the path

pkg/gui/gui.go

@@ -448,14 +448,15 @@ func NewGui(
 		// sake of backwards compatibility. We're making use of short circuiting here
 		ShowExtrasWindow: cmn.UserConfig.Gui.ShowCommandLog && !config.GetAppState().HideCommandLog,
 		Mutexes: types.Mutexes{
-			RefreshingFilesMutex:  &deadlock.Mutex{},
-			RefreshingStatusMutex: &deadlock.Mutex{},
-			SyncMutex:             &deadlock.Mutex{},
-			LocalCommitsMutex:     &deadlock.Mutex{},
-			SubCommitsMutex:       &deadlock.Mutex{},
-			SubprocessMutex:       &deadlock.Mutex{},
-			PopupMutex:            &deadlock.Mutex{},
-			PtyMutex:              &deadlock.Mutex{},
+			RefreshingFilesMutex:    &deadlock.Mutex{},
+			RefreshingBranchesMutex: &deadlock.Mutex{},
+			RefreshingStatusMutex:   &deadlock.Mutex{},
+			SyncMutex:               &deadlock.Mutex{},
+			LocalCommitsMutex:       &deadlock.Mutex{},
+			SubCommitsMutex:         &deadlock.Mutex{},
+			SubprocessMutex:         &deadlock.Mutex{},
+			PopupMutex:              &deadlock.Mutex{},
+			PtyMutex:                &deadlock.Mutex{},
 		},
 		InitialDir: initialDir,
 	}

pkg/gui/types/common.go

@@ -217,14 +217,15 @@ type Model struct {
 // if you add a new mutex here be sure to instantiate it. We're using pointers to
 // mutexes so that we can pass the mutexes to controllers.
 type Mutexes struct {
-	RefreshingFilesMutex  *deadlock.Mutex
-	RefreshingStatusMutex *deadlock.Mutex
-	SyncMutex             *deadlock.Mutex
-	LocalCommitsMutex     *deadlock.Mutex
-	SubCommitsMutex       *deadlock.Mutex
-	SubprocessMutex       *deadlock.Mutex
-	PopupMutex            *deadlock.Mutex
-	PtyMutex              *deadlock.Mutex
+	RefreshingFilesMutex    *deadlock.Mutex
+	RefreshingBranchesMutex *deadlock.Mutex
+	RefreshingStatusMutex   *deadlock.Mutex
+	SyncMutex               *deadlock.Mutex
+	LocalCommitsMutex       *deadlock.Mutex
+	SubCommitsMutex         *deadlock.Mutex
+	SubprocessMutex         *deadlock.Mutex
+	PopupMutex              *deadlock.Mutex
+	PtyMutex                *deadlock.Mutex
 }
 
 type IStateAccessor interface {