Add a class to replace ImageLoadingTracker with a nicer API.

chrome/browser/extensions/image_loader_factory.h

Index: chrome/browser/extensions/image_loader_factory.h
diff --git a/chrome/browser/extensions/app_restore_service_factory.h b/chrome/browser/extensions/image_loader_factory.h
similarity index 51%
copy from chrome/browser/extensions/app_restore_service_factory.h
copy to chrome/browser/extensions/image_loader_factory.h
index 519e23425c618d55791748bced7c905e076a9f0a..767316b3e39ee97b817598b97108150af7822398 100644
--- a/chrome/browser/extensions/app_restore_service_factory.h
+++ b/chrome/browser/extensions/image_loader_factory.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_EXTENSIONS_APP_RESTORE_SERVICE_FACTORY_H_
-#define CHROME_BROWSER_EXTENSIONS_APP_RESTORE_SERVICE_FACTORY_H_
+#ifndef CHROME_BROWSER_EXTENSIONS_IMAGE_LOADER_FACTORY_H_
+#define CHROME_BROWSER_EXTENSIONS_IMAGE_LOADER_FACTORY_H_
 
 #include "base/memory/singleton.h"
 #include "chrome/browser/profiles/profile_keyed_service_factory.h"
@@ -12,31 +12,32 @@ class Profile;
 
 namespace extensions {
 
-class AppRestoreService;
+class ImageLoader;

[Aaron Boodman, 2012/11/08 21:07:22]

Can you please create a class template for this in like chrome/browser/profiles?
It pains me to see so many copies of this pattern.

 
-// Singleton that owns all AppRestoreServices and associates them with
+// Singleton that owns all ImageLoaders and associates them with
 // Profiles. Listens for the Profile's destruction notification and cleans up
-// the associated AppRestoreService.
-class AppRestoreServiceFactory : public ProfileKeyedServiceFactory {
+// the associated ImageLoader.
+class ImageLoaderFactory : public ProfileKeyedServiceFactory {
  public:
-  static AppRestoreService* GetForProfile(Profile* profile);
+  static ImageLoader* GetForProfile(Profile* profile);
 
   static void ResetForProfile(Profile* profile);
 
-  static AppRestoreServiceFactory* GetInstance();
+  static ImageLoaderFactory* GetInstance();
 
  private:
-  friend struct DefaultSingletonTraits<AppRestoreServiceFactory>;
+  friend struct DefaultSingletonTraits<ImageLoaderFactory>;
 
-  AppRestoreServiceFactory();
-  virtual ~AppRestoreServiceFactory();
+  ImageLoaderFactory();
+  virtual ~ImageLoaderFactory();
 
   // ProfileKeyedServiceFactory:
   virtual ProfileKeyedService* BuildServiceInstanceFor(
       Profile* profile) const OVERRIDE;
   virtual bool ServiceIsCreatedWithProfile() const OVERRIDE;
+  virtual bool ServiceRedirectedInIncognito() const OVERRIDE;
 };
 
 }  // namespace extensions
 
-#endif  // CHROME_BROWSER_EXTENSIONS_APP_RESTORE_SERVICE_FACTORY_H_
+#endif  // CHROME_BROWSER_EXTENSIONS_IMAGE_LOADER_FACTORY_H_