Close wrench menu when Media Router Action clicked in Views.

chrome/browser/ui/views/media_router/media_router_ui_browsertest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
 #include "chrome/browser/ui/toolbar/media_router_action.h"
 #include "chrome/browser/ui/toolbar/toolbar_action_view_delegate.h"
 #include "chrome/browser/ui/views/frame/browser_view.h"
@@ skipping 15 matching lines @@
 
   void SetUpOnMainThread() override {
     InProcessBrowserTest::SetUpOnMainThread();
 
     BrowserActionsContainer* browser_actions_container =
         BrowserView::GetBrowserViewForBrowser(browser())
             ->toolbar()
             ->browser_actions();
     ASSERT_TRUE(browser_actions_container);
 
-    media_router_action_.reset(new MediaRouterAction);
+    media_router_action_.reset(new MediaRouterAction(browser()));
 
     // Sets delegate on |media_router_action_|.
     toolbar_action_view_.reset(
         new ToolbarActionView(media_router_action_.get(), browser()->profile(),
                               browser_actions_container));
   }
 
   void OpenMediaRouterDialogAndWaitForNewWebContents() {
     content::TestNavigationObserver nav_observer(NULL);
     nav_observer.StartWatchingNewWebContents();
     media_router_action_->ExecuteAction(true);
+
+    ToolbarView* toolbar =
+        BrowserView::GetBrowserViewForBrowser(browser())->toolbar();
+    ASSERT_FALSE(toolbar->IsWrenchMenuShowing());

[Devlin, 2015/07/14 21:31:07]

... But the wrench menu is never opened, right?  Won't this be trivially true?

[apacible, 2015/07/16 16:52:11]

... good point. Opening the wrench menu prior to the ExecuteAction call now...

+
     nav_observer.Wait();
     nav_observer.StopWatchingNewWebContents();
   }
 
  protected:
   // Must be initialized after |InProcessBrowserTest::SetUpOnMainThread|.
   scoped_ptr<MediaRouterAction> media_router_action_;
 
   // ToolbarActionView constructed to set the delegate on |mr_action|.
   scoped_ptr<ToolbarActionView> toolbar_action_view_;
@@ skipping 22 matching lines @@
       browser()->tab_strip_model()->GetActiveWebContents(), 1);
   ui_test_utils::NavigateToURL(browser(), GURL("about:blank"));
   nav_observer.Wait();
 
   // The navigation should have removed the previously created dialog.
   // We expect a new dialog WebContents to be created by calling this.
   OpenMediaRouterDialogAndWaitForNewWebContents();
 }
 
 }  // namespace media_router