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

chrome/browser/importer/importer_host.h

 // Copyright (c) 2011 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_IMPORTER_IMPORTER_HOST_H_
 #define CHROME_BROWSER_IMPORTER_IMPORTER_HOST_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/bookmarks/base_bookmark_model_observer.h"
 #include "chrome/browser/importer/importer_data_types.h"
 #include "chrome/browser/importer/profile_writer.h"
-#include "chrome/browser/ui/browser_list.h"
+#include "chrome/browser/ui/browser_list_observer.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "ui/gfx/native_widget_types.h"
 
 class FirefoxProfileLock;
 class Importer;
 class Profile;
 
 namespace importer {
 class ImporterProgressObserver;
 }
 
 // This class hosts the importers. It enumerates profiles from other
 // browsers dynamically, and controls the process of importing. When
 // the import process is done, ImporterHost deletes itself.
 class ImporterHost : public base::RefCountedThreadSafe<ImporterHost>,
                      public BaseBookmarkModelObserver,
                      public content::NotificationObserver,
-                     public BrowserList::Observer {
+                     public chrome::BrowserListObserver {
  public:
   ImporterHost();
 
   // ShowWarningDialog() asks user to close the application that is owning the
   // lock. They can retry or skip the importing process.
   void ShowWarningDialog();
 
   // This is called when when user ends the lock dialog by clicking on either
   // the "Skip" or "Continue" buttons. |is_continue| is true when user clicked
   // the "Continue" button.
@@ skipping 89 matching lines @@
   virtual void Loaded(BookmarkModel* model, bool ids_reassigned) OVERRIDE;
   virtual void BookmarkModelBeingDeleted(BookmarkModel* model) OVERRIDE;
   virtual void BookmarkModelChanged() OVERRIDE;
 
   // content::NotificationObserver:
   // Called when TemplateURLService has been loaded.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
-  // BrowserList::Observer
+  // chrome::BrowserListObserver
   virtual void OnBrowserRemoved(Browser* browser) OVERRIDE;
 
   // The task is the process of importing settings from other browsers.
   base::Closure task_;
 
   // The importer used in the task.
   Importer* importer_;
 
   // True if UI is not to be shown.
   bool headless_;
 
   // Parent window that we pass to the import lock dialog (i.e, the Firefox
   // warning dialog).
   gfx::NativeWindow parent_window_;
 
   // Used to add a new tab if we need the user to sign in.
   Browser* browser_;
 
   // The observer that we need to notify about changes in the import process.
   importer::ImporterProgressObserver* observer_;
 
   // Firefox profile lock.
   scoped_ptr<FirefoxProfileLock> firefox_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(ImporterHost);
 };
 
 #endif  // CHROME_BROWSER_IMPORTER_IMPORTER_HOST_H_