Action box menu

chrome/browser/extensions/extension_toolbar_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_EXTENSIONS_EXTENSION_TOOLBAR_MODEL_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_TOOLBAR_MODEL_H_
 
 #include "base/compiler_specific.h"
 #include "base/observer_list.h"
 #include "chrome/common/extensions/extension.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 
 class Browser;
 class ExtensionService;
 class PrefService;
 
 // Model for the browser actions toolbar.
 class ExtensionToolbarModel : public content::NotificationObserver {
  public:
+  typedef std::vector<std::string> VectorOfStrings;

[Peter Kasting, 2012/07/21 01:55:13]

Nit: As noted in extension_prefs.h, if you change the functions there to
consistently use the pre-existing ExtensionIdSet type, you can eliminate this
typedef and use that type instead in this header/.cc file.  (This will require
#including extension_prefs.h here, which seems OK to me.)

[yefimt, 2012/07/23 23:47:52]

Done.

+
   explicit ExtensionToolbarModel(ExtensionService* service);
   virtual ~ExtensionToolbarModel();
 
   // The action that should be taken as a result of clicking a browser action.
   enum Action {
     ACTION_NONE,
     ACTION_SHOW_POPUP,
     // Unlike LocationBarController there is no ACTION_SHOW_CONTEXT_MENU,
     // because UI implementations tend to handle this themselves at a higher
     // level.
@@ skipping 35 matching lines @@
                               GURL* popup_url_out);
   // If count == size(), this will set the visible icon count to -1, meaning
   // "show all actions".
   void SetVisibleIconCount(int count);
   // As above, a return value of -1 represents "show all actions".
   int GetVisibleIconCount() const { return visible_icon_count_; }
 
   bool extensions_initialized() const { return extensions_initialized_; }
 
   size_t size() const {
-    return toolitems_.size();
+    return toolbar_items_.size();
   }
 
   extensions::ExtensionList::iterator begin() {
-    return toolitems_.begin();
+    return toolbar_items_.begin();
   }
 
   extensions::ExtensionList::iterator end() {
-    return toolitems_.end();
+    return toolbar_items_.end();
   }
 
-  const extensions::Extension* GetExtensionByIndex(int index) const;
+  const extensions::Extension* GetExtensionByIndex(int index) const {

[Peter Kasting, 2012/07/21 01:55:13]

Nit: Inlined functions should be named unix_hacker()-style.

Normally we'd also name them without "get" and as close to the actual variable
names in question as possible, a la:

toolbar_item_for_index()
action_box_menu_item_for_index()

(2 places)

[yefimt, 2012/07/23 23:47:52]

Yes, but they not returning a member variable but and expression with it, and
renaming this function will cause a chain reaction:)
I'll need to touch 2 other files, then you will ask me to fix them and because
they are mac and linux specific, I cannot even verify they compile.

[Peter Kasting, 2012/07/24 04:27:29]

That's what we have trybots for.  Please do at least make the name use lowercase
and underscores, the actual names are up to your best judgment.

+    return toolbar_items_[index];
+  }
 
   // Utility functions for converting between an index into the list of
   // incognito-enabled browser actions, and the list of all browser actions.
   int IncognitoIndexToOriginal(int incognito_index);
   int OriginalIndexToIncognito(int original_index);
 
+  size_t action_box_extensions_size() {
+    return action_box_menu_items_.size();
+  }
+
+  const extensions::Extension* GetActionBoxExtensionByIndex(int index) const {
+    return action_box_menu_items_[index];
+  }
+
  private:
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
   // To be called after the extension service is ready; gets loaded extensions
   // from the extension service and their saved order from the pref service
-  // and constructs |toolitems_| from these data.
-  void InitializeExtensionList();
+  // and constructs |toolbar_items_| and |action_box_items_| from these data.
+  void InitializeExtensionLists();
+  void PopulateForActionBoxMode();
+  void PopulateForNonActionBoxMode();
+
+  // A couple helper functions to populate lists.
+  void AddToProperList(const extensions::Extension* extension,
+                       const VectorOfStrings& order,
+                       extensions::ExtensionList* sorted,
+                       extensions::ExtensionList* unsorted);
+  void MergeLists(const extensions::ExtensionList& sorted,
+                  const extensions::ExtensionList& unsorted,
+                  extensions::ExtensionList* result_list);
+  void FillExtensionList(const VectorOfStrings& order,
+                         extensions::ExtensionList* result_list);
 
   // Save the model to prefs.
   void UpdatePrefs();
 
   // Our observers.
   ObserverList<Observer> observers_;
 
   void AddExtension(const extensions::Extension* extension);
   void RemoveExtension(const extensions::Extension* extension);
 
   // Our ExtensionService, guaranteed to outlive us.
   ExtensionService* service_;
 
   PrefService* prefs_;
 
   // True if we've handled the initial EXTENSIONS_READY notification.
   bool extensions_initialized_;
 
   // Ordered list of browser action buttons.
-  extensions::ExtensionList toolitems_;
+  extensions::ExtensionList toolbar_items_;
+
+  // List of browser action buttons visible in an action box menu.
+  extensions::ExtensionList action_box_menu_items_;
 
   // Keeps track of what the last extension to get disabled was.
   std::string last_extension_removed_;
 
   // Keeps track of where the last extension to get disabled was in the list.
   size_t last_extension_removed_index_;
 
   // The number of icons visible (the rest should be hidden in the overflow
   // chevron).
   int visible_icon_count_;
 
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionToolbarModel);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_TOOLBAR_MODEL_H_