Action box menu

chrome/browser/extensions/extension_prefs.h

Index: chrome/browser/extensions/extension_prefs.h
diff --git a/chrome/browser/extensions/extension_prefs.h b/chrome/browser/extensions/extension_prefs.h
index 51d61168aba35660e9020487ff8b7f820664d04b..1e14d1ce922b2f462d8c80dcfc09b713deac1dd7 100644
--- a/chrome/browser/extensions/extension_prefs.h
+++ b/chrome/browser/extensions/extension_prefs.h
@@ -108,6 +108,10 @@ class ExtensionPrefs : public ContentSettingsStore::Observer,
   // Set the order that the browser actions appear in the toolbar.

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

Nit: Eliminate this comment and the blank line above in favor of using "Get/Set"
like you have in the comment for [Get,Set]ActionBoxOrder().

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

Done.

   void SetToolbarOrder(const std::vector<std::string>& extension_ids);
 
+  // Get/Set the order that the browser actions appear in the action box.
+  std::vector<std::string> GetActionBoxOrder();
+  void SetActionBoxOrder(const std::vector<std::string>& extension_ids);
+
   // Called when an extension is installed, so that prefs get created.
   // If |page_ordinal| is an invalid ordinal, then a page will be found
   // for the App.
@@ -544,6 +548,12 @@ class ExtensionPrefs : public ContentSettingsStore::Observer,
   bool DoesExtensionHaveState(const std::string& id,
                               Extension::State check_state) const;
 
+  // Helper function to Get/Set array of strings from/to prefs.
+  std::vector<std::string> GetExtensionPrefAsVector(const char* pref);
+  void SetExtensionPrefFromVector(
+      const char* pref,
+      const std::vector<std::string>& extension_ids);
+
   // The pref service specific to this set of extension prefs. Owned by profile.
   PrefService* prefs_;