Close wrench menu when Media Router Action clicked in Views.

chrome/browser/ui/toolbar/media_router_action_platform_delegate.h

Index: chrome/browser/ui/toolbar/media_router_action_platform_delegate.h
diff --git a/chrome/browser/ui/toolbar/media_router_action_platform_delegate.h b/chrome/browser/ui/toolbar/media_router_action_platform_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..5da94d635fcabbada2f1c3c8f5e1500f5add8c9c
--- /dev/null
+++ b/chrome/browser/ui/toolbar/media_router_action_platform_delegate.h
@@ -0,0 +1,23 @@
+// 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.
+
+#ifndef CHROME_BROWSER_UI_TOOLBAR_MEDIA_ROUTER_ACTION_PLATFORM_DELEGATE_H_
+#define CHROME_BROWSER_UI_TOOLBAR_MEDIA_ROUTER_ACTION_PLATFORM_DELEGATE_H_
+
+#include "base/memory/scoped_ptr.h"
+#include "chrome/browser/ui/browser.h"

[Devlin, 2015/07/14 17:23:02]

Forward declaration works.

[apacible, 2015/07/14 21:15:23]

Done.

+
+class MediaRouterActionPlatformDelegate {
+ public:
+  virtual ~MediaRouterActionPlatformDelegate() {}
+
+  // Returns a created MediaRouterActionPlatformDelegate. This is defined in the
+  // platform-specific implementation for the class.
+  static scoped_ptr<MediaRouterActionPlatformDelegate> Create(Browser* browser);
+
+  // Closes the overflow menu, if it was open.

[Devlin, 2015/07/14 17:23:02]

The comment is good, but I'd rename the method to be CloseOverflowMenuIfOpen()
just to make it clear when reading the code.

[apacible, 2015/07/14 21:15:23]

Done.

+  virtual void CloseOverflowMenu() = 0;
+};
+
+#endif  // CHROME_BROWSER_UI_TOOLBAR_MEDIA_ROUTER_ACTION_PLATFORM_DELEGATE_VIEWS_H_