Only show chrome2mobile in action box if it is enabled

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 "chrome/browser/extensions/extension_service.h"

[msw, 2012/08/30 01:47:01]

nit: use chrome/common/extensions/extension.h instead (needed for const
extensions::ExtensionList&) and you can remove the Extension forward decl below.

[Cait (Slow), 2012/08/30 21:46:03]

Done.

 #include "content/public/browser/notification_observer.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/ui/browser_commands.h"
 #include "ui/base/models/simple_menu_model.h"
 
 class Browser;
 
 namespace extensions {
 class Extension;

[msw, 2012/08/30 01:47:01]

nit: remove this when including extension.h instead of extension_service.h

[Cait (Slow), 2012/08/30 21:46:03]

Done.

 }
 
 // A menu model that builds the contents of the action box menu. This model
 // should be built on demand since its content reflects the state of the browser
 // at creation time.
 class ActionBoxMenuModel : public ui::SimpleMenuModel,
                            public ui::SimpleMenuModel::Delegate,
                            public content::NotificationObserver {
  public:
-  ActionBoxMenuModel(Browser* browser,
-                     ExtensionService* extension_service);
+  explicit ActionBoxMenuModel(Browser* browser);
   virtual ~ActionBoxMenuModel();
 
   // Returns true if item associated with an extension.
   bool IsItemExtension(int index);
 
   // Returns an extension associated with model item at |index|
   // or NULL if it is not an extension item.
   const extensions::Extension* GetExtensionAt(int index);
 
   // Overridden from ui::SimpleMenuModel::Delegate:
   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;
   virtual void ExecuteCommand(int command_id) OVERRIDE;
 
  private:
-  const extensions::ExtensionList& action_box_menu_items() {
-    return extension_service_->toolbar_model()->action_box_menu_items();
-  }
+  const extensions::ExtensionList& GetActionBoxMenuItems();
 
   typedef std::map<int, std::string> IdToEntensionIdMap;
 
   // Overridden from content::NotificationObserver:
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   Browser* browser_;
-  ExtensionService* extension_service_;
 
   IdToEntensionIdMap id_to_extension_id_map_;
 
   DISALLOW_COPY_AND_ASSIGN(ActionBoxMenuModel);
 };
 
 #endif  // CHROME_BROWSER_UI_TOOLBAR_ACTION_BOX_MENU_MODEL_H_