Merge pull request #23367 from yashykt/130fixws

Fix possible deadlock in RemoveExternalConnectivityWatcher

src/core/ext/filters/client_channel/client_channel.cc

@@ -179,12 +179,16 @@ class ChannelData {
 
   void RemoveExternalConnectivityWatcher(grpc_closure* on_complete,
                                          bool cancel) {
-    MutexLock lock(&external_watchers_mu_);
+    ExternalConnectivityWatcher* watcher = nullptr;
-    auto it = external_watchers_.find(on_complete);
+    {
-    if (it != external_watchers_.end()) {
+      MutexLock lock(&external_watchers_mu_);
-      if (cancel) it->second->Cancel();
+      auto it = external_watchers_.find(on_complete);
-      external_watchers_.erase(it);
+      if (it != external_watchers_.end()) {
+        watcher = it->second;
+        external_watchers_.erase(it);
+      }
     }
+    if (watcher != nullptr && cancel) watcher->Cancel();
   }
 
   int NumExternalConnectivityWatchers() const {