Extension Toolbar redesign, part 1 (overflow)

chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h

Index: chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
diff --git a/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
new file mode 100644
index 0000000000000000000000000000000000000000..26003fe6bd5e1f4f5eeb9143876b341b8b6b9ac2
--- /dev/null
+++ b/chrome/browser/ui/views/toolbar/extension_toolbar_menu_view.h
@@ -0,0 +1,31 @@
+// Copyright 2014 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_VIEWS_TOOLBAR_EXTENSION_TOOLBAR_MENU_VIEW_H_
+#define CHROME_BROWSER_UI_VIEWS_TOOLBAR_EXTENSION_TOOLBAR_MENU_VIEW_H_
+
+#include "ui/views/view.h"
+
+class Browser;
+class BrowserActionsContainer;
+
+// ExtensionToolbarMenuView is the view containing the ToolbarAction icons.

[Devlin, 2014/06/23 17:55:29]

This comment doesn't seem quite right.  "ToolbarAction" is vague, and this
should mention that this is owned by the wrench menu.  "ExtensionToolbarMenuView
is the view containing the extension actions that overflowed from the
BrowserActionsContainer, and is contained in and owned by the wrench menu." or
something to that effect?

[Finnur, 2014/06/25 16:18:11]

SGTM

+class ExtensionToolbarMenuView : public views::View {
+ public:
+  explicit ExtensionToolbarMenuView(Browser* browser);
+  virtual ~ExtensionToolbarMenuView();
+
+  // Overridden from View.
+  virtual gfx::Size GetPreferredSize() const OVERRIDE;
+  virtual void Layout() OVERRIDE;
+
+ private:
+  Browser* browser_;
+
+  BrowserActionsContainer* container_;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionToolbarMenuView);
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_TOOLBAR_EXTENSION_TOOLBAR_MENU_VIEW_H_