Refactor implementation of BrowserActions, and add support for

chrome/browser/extensions/extensions_service.h

 // Copyright (c) 2009 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_EXTENSIONS_SERVICE_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSIONS_SERVICE_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 88 matching lines @@
 
   // Look up an extension by ID.
   Extension* GetExtensionById(const std::string& id) {
     return GetExtensionByIdInternal(id, true, false);
   }
 
   // Retrieves a vector of all page actions, irrespective of which extension
   // 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
   // immediately loaded.
   // TODO(aa): This method can be removed. It is only used by the unit tests,
   // and they could use CrxInstaller directly instead.
   void InstallExtension(const FilePath& extension_path);
 
   // Updates a currently-installed extension with the contents from
   // |extension_path|.
@@ skipping 119 matching lines @@
   void LoadInstalledExtension(
       DictionaryValue* manifest, const std::string& id,
       const FilePath& path, Extension::Location location);
 
   // Handles sending notification that |extension| was loaded.
   void NotifyExtensionLoaded(Extension* extension);
 
   // 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_;
 
   // Preferences for the owning profile.
   scoped_ptr<ExtensionPrefs> extension_prefs_;
 
   // The message loop to use with the backend.
   MessageLoop* backend_loop_;
 
   // The current list of installed extensions.
@@ skipping 117 matching lines @@
 
   // A map of all external extension providers.
   typedef std::map<Extension::Location,
                    linked_ptr<ExternalExtensionProvider> > ProviderMap;
   ProviderMap external_extension_providers_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionsServiceBackend);
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSIONS_SERVICE_H_