Move implementation of BrowserList onto an inner, instantiatable class, BrowserListImpl.

chrome/browser/extensions/bundle_installer.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_BUNDLE_INSTALLER_H_
 #define CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_
 #pragma once
 
 #include <string>
 #include <vector>
 
 #include "base/memory/linked_ptr.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/extensions/webstore_installer.h"
 #include "chrome/browser/extensions/webstore_install_helper.h"
-#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_list_observer.h"
 #include "chrome/common/extensions/extension.h"
 
 namespace base {
 class DictionaryValue;
 }  // namespace base
 
 namespace content {
 class NavigationController;
 }  // namespace content
 
 class Browser;
 class Profile;
 
 namespace extensions {
 
 // Manages the installation life cycle for extension bundles.
 //
 // We install bundles in two steps:
 //  1) PromptForApproval: parse manifests and prompt the user
 //  2) CompleteInstall: install the CRXs and show confirmation bubble
 //
 class BundleInstaller : public WebstoreInstallHelper::Delegate,
                         public ExtensionInstallPrompt::Delegate,
                         public WebstoreInstaller::Delegate,
-                        public BrowserList::Observer,
+                        public chrome::BrowserListObserver,
                         public base::RefCountedThreadSafe<BundleInstaller> {
  public:
   // Auto approve or cancel the permission prompt.
   static void SetAutoApproveForTesting(bool approve);
 
   class Delegate {
    public:
     virtual void OnBundleInstallApproved() {}
     virtual void OnBundleInstallCanceled(bool user_initiated) {}
     virtual void OnBundleInstallCompleted() {}
@@ skipping 104 matching lines @@
 
   // ExtensionInstallPrompt::Delegate implementation:
   virtual void InstallUIProceed() OVERRIDE;
   virtual void InstallUIAbort(bool user_initiated) OVERRIDE;
 
   // WebstoreInstaller::Delegate implementation:
   virtual void OnExtensionInstallSuccess(const std::string& id) OVERRIDE;
   virtual void OnExtensionInstallFailure(const std::string& id,
                                          const std::string& error) OVERRIDE;
 
-  // BrowserList::observer implementation:
+  // chrome::BrowserListObserver implementation:
   virtual void OnBrowserAdded(Browser* browser) OVERRIDE;
   virtual void OnBrowserRemoved(Browser* browser) OVERRIDE;
   virtual void OnBrowserSetLastActive(Browser* browser) OVERRIDE;
 
   // Holds the Extensions used to generate the permission warnings.
   ExtensionList dummy_extensions_;
 
   // Holds the parsed manifests, indexed by the extension ids.
   ManifestMap parsed_manifests_;
 
@@ skipping 13 matching lines @@
   scoped_ptr<ExtensionInstallPrompt> install_ui_;
 
   Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(BundleInstaller);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_BUNDLE_INSTALLER_H_