On Mac: Hide any popup menus on the active view before switching to another.

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 41a767e7da3fe91aac8fa5579ed60d5240b79bb7..79ec93d21d314ef9a045106a1e263f418539f33d 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1073,6 +1073,12 @@ void WebContentsImpl::WasHidden() {
 
   FOR_EACH_OBSERVER(WebContentsObserver, observers_, WasHidden());
 
+#if defined(OS_MACOSX)
+  RenderViewHostDelegateView* view = GetDelegateView();
+  if (view)
+    view->HidePopupMenu();
+#endif

[Avi (use Gerrit), 2015/01/06 15:46:39]

Android also uses external popup menus; do we want to do this for them?

Also, I'm very suspicious of this code being in this file.
ShowPopupMenu/HidePopupMenu calls are made in RenderFrameHostImpl. Why should
this code go here?

+
   should_normally_be_visible_ = false;
 }