Make sure the ActiveRepo data is always up to date

If the origin remote url changes, update ActiveRepo that as soon as
possible.

Author: shana

src/GitHub.Exports/Services/ITeamExplorerServiceHolder.cs

@@ -43,6 +43,11 @@ public interface ITeamExplorerServiceHolder
         void Unsubscribe(object who);
 
         IGitAwareItem HomeSection { get; }
+
+        /// <summary>
+        /// Refresh the information on the active repo (in case of remote url changes or other such things)
+        /// </summary>
+        void Refresh();
     }
 
     public interface IGitAwareItem

src/GitHub.VisualStudio/Base/TeamExplorerServiceHolder.cs

@@ -59,25 +59,40 @@ public ISimpleRepositoryModel ActiveRepo
             [return: AllowNull] get { return activeRepo; }
             private set
             {
-                if (Equals(activeRepo, value))
+                if (activeRepo == value)
                     return;
+                if (activeRepo != null)
+                    activeRepo.PropertyChanged -= ActiveRepoPropertyChanged;
                 activeRepo = value;
+                if (activeRepo != null)
+                    activeRepo.PropertyChanged += ActiveRepoPropertyChanged;
                 NotifyActiveRepo();
             }
         }
 
         public void Subscribe(object who, Action<ISimpleRepositoryModel> handler)
         {
+            bool notificationsExist;
+            ISimpleRepositoryModel repo;
             lock(activeRepoHandlers)
             {
-                var repo = ActiveRepo;
+                repo = ActiveRepo;
+                notificationsExist = activeRepoNotified;
                 if (!activeRepoHandlers.ContainsKey(who))
                     activeRepoHandlers.Add(who, handler);
                 else
                     activeRepoHandlers[who] = handler;
-                if (activeRepoNotified)
-                    handler(repo);
             }
+
+            // the repo url might have changed and we don't get notifications
+            // for that, so this is a good place to refresh it in case that happened
+            repo?.Refresh();
+
+            // if the active repo hasn't changed and there's notifications queued up,
+            // notify the subscriber. If the repo has changed, the set above will trigger
+            // notifications so we don't have to do it here.
+            if (repo == ActiveRepo && notificationsExist)
+                handler(repo);
         }
 
         public void Unsubscribe(object who)
@@ -101,6 +116,12 @@ public void ClearServiceProvider(IServiceProvider provider)
             ServiceProvider = null;
         }
 
+        public void Refresh()
+        {
+            GitUIContext = GitUIContext ?? UIContext.FromUIContextGuid(new Guid("11B8E6D7-C08B-4385-B321-321078CDD1F8"));
+            UIContextChanged(GitUIContext?.IsActive ?? false, true);
+        }
+
         void NotifyActiveRepo()
         {
             lock (activeRepoHandlers)
@@ -114,10 +135,10 @@ void NotifyActiveRepo()
         void UIContextChanged(object sender, UIContextChangedEventArgs e)
         {
             ActiveRepo = null;
-            UIContextChanged(e.Activated);
+            UIContextChanged(e.Activated, false);
         }
 
-        async void UIContextChanged(bool active)
+        async void UIContextChanged(bool active, bool refresh)
         {
             Debug.Assert(ServiceProvider != null, "UIContextChanged called before service provider is set");
             if (ServiceProvider == null)
@@ -126,7 +147,7 @@ async void UIContextChanged(bool active)
             if (active)
             {
                 GitService = GitService ?? ServiceProvider.GetService<IGitExt>();
-                if (ActiveRepo == null)
+                if (ActiveRepo == null || refresh)
                     ActiveRepo = await System.Threading.Tasks.Task.Run(() =>
                     {
                         var repos = GitService?.ActiveRepositories;
@@ -158,12 +179,17 @@ void CheckAndUpdate(object sender, System.ComponentModel.PropertyChangedEventArg
                 return;
 
             var repo = service.ActiveRepositories.FirstOrDefault()?.ToModel();
-            // this comparison is safe, the extension method supports null instances
-            if (!repo.Equals(ActiveRepo))
+            if (repo != ActiveRepo)
                 // so annoying that this is on the wrong thread
                 syncContext.Post(r => ActiveRepo = r as ISimpleRepositoryModel, repo);
         }
 
+        void ActiveRepoPropertyChanged(object sender, System.ComponentModel.PropertyChangedEventArgs e)
+        {
+            if (e.PropertyName == "CloneUrl")
+                ActiveRepo = sender as ISimpleRepositoryModel;
+        }
+
         public IGitAwareItem HomeSection
         {
             [return:AllowNull]