Action box menu

chrome/browser/ui/toolbar/action_box_menu_model.h

+// Copyright (c) 2012 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_ACTION_BOX_MENU_MODEL_H_
+#define CHROME_BROWSER_UI_TOOLBAR_ACTION_BOX_MENU_MODEL_H_
+
+#include <map>
+#include "base/file_path.h"

[Peter Kasting, 2012/07/14 02:08:01]

Nit: Add blank line here

[yefimt, 2012/07/17 18:20:37]

Done.

+#include "base/memory/scoped_ptr.h"
+#include "content/public/browser/notification_observer.h"
+#include "ui/base/models/button_menu_item_model.h"
+#include "ui/base/models/simple_menu_model.h"
+
+class ExtensionService;
+
+namespace extensions {
+class Extension;
+}
+
+
+// A menu model that builds the contents of the action box menu.
+class ActionBoxMenuModel : public ui::SimpleMenuModel,
+                           public ui::SimpleMenuModel::Delegate,
+                           public content::NotificationObserver {
+ public:
+  explicit ActionBoxMenuModel(ExtensionService* extension_service);
+  virtual ~ActionBoxMenuModel();
+
+  // Active box extensions.
+  size_t action_box_extensions_size();
+  const extensions::Extension* GetActionBoxExtensionByIndex(int index);
+
+ private:
+   // Overridden for both ButtonMenuItemModel::Delegate and SimpleMenuModel
+  virtual void ExecuteCommand(int command_id) OVERRIDE;
+  virtual bool IsCommandIdChecked(int command_id) const OVERRIDE;
+  virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE;
+  virtual bool GetAcceleratorForCommandId(
+      int command_id, ui::Accelerator* accelerator) OVERRIDE;

[Peter Kasting, 2012/07/14 02:08:01]

Nit: One arg per line

[yefimt, 2012/07/17 18:20:37]

Done.

+
+  // Overridden from content::NotificationObserver:
+  virtual void Observe(int type,
+                       const content::NotificationSource& source,
+                       const content::NotificationDetails& details) OVERRIDE;
+
+  void Build();

[Peter Kasting, 2012/07/14 02:08:01]

What does this do?

[yefimt, 2012/07/17 18:20:37]

Done.

+
+  ExtensionService* extension_service_;
+
+  typedef std::map<int, std::string> IdToEntensionId;

[Peter Kasting, 2012/07/14 02:08:01]

Nit: typedef goes atop the private section.  Put "Map" in the name.

[yefimt, 2012/07/17 18:20:37]

Done.

+  IdToEntensionId id_to_extension_id_;
+
+  DISALLOW_COPY_AND_ASSIGN(ActionBoxMenuModel);
+};
+
+#endif  // CHROME_BROWSER_UI_TOOLBAR_ACTION_BOX_MENU_MODEL_H_