Refactor implementation of BrowserActions, and add support for

chrome/browser/extensions/extensions_service.h

Index: chrome/browser/extensions/extensions_service.h
diff --git a/chrome/browser/extensions/extensions_service.h b/chrome/browser/extensions/extensions_service.h
index 4235c22f6b2c4f95a420657b6efa94915e574bbb..0dad4cd2619c99fae27483f8dceae37b4a95048a 100644
--- a/chrome/browser/extensions/extensions_service.h
+++ b/chrome/browser/extensions/extensions_service.h
@@ -106,10 +106,6 @@ class ExtensionsService
   // they belong to.
   std::vector<ExtensionAction*> GetPageActions() const;
 
-  // Retrieves a vector of all browser actions, irrespective of which extension
-  // they belong to.
-  std::vector<ExtensionAction*> GetBrowserActions(bool include_popups) const;
-
   // Install the extension file at |extension_path|.  Will install as an
   // update if an older version is already installed.
   // For fresh installs, this method also causes the extension to be
@@ -249,13 +245,6 @@ class ExtensionsService
   // Handles sending notification that |extension| was unloaded.
   void NotifyExtensionUnloaded(Extension* extension);
 
-  // Retrieves a vector of all page actions or browser actions, irrespective of
-  // which extension they belong to.  If |include_popups| is false, actions that
-  // are popups are excluded.
-  std::vector<ExtensionAction*> GetExtensionActions(
-      ExtensionAction::ExtensionActionType action_type,
-      bool include_popups) const;
-
   // The profile this ExtensionsService is part of.
   Profile* profile_;