Refactored loading of applications / extensions icons.

chrome/browser/ui/webui/extensions/extension_icon_source.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_UI_WEBUI_EXTENSIONS_EXTENSION_ICON_SOURCE_H_
 #define CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_ICON_SOURCE_H_
 
 #include <map>
 #include <string>
 
@@ skipping 66 matching lines @@
       int render_process_id,
       int render_view_id,
       const content::URLDataSource::GotDataCallback& callback) OVERRIDE;
 
  private:
   // Encapsulates the request parameters for |request_id|.
   struct ExtensionIconRequest;
 
   virtual ~ExtensionIconSource();
 
-  // Returns the bitmap for the default app image.
-  const SkBitmap* GetDefaultAppImage();
+  // Loads the extension's icon for the given |request_id| and returns the
+  // image to the client.
+  void LoadExtensionImage(int request_id);
 
-  // Returns the bitmap for the default extension.
-  const SkBitmap* GetDefaultExtensionImage();
-
-  // Performs any remaining transformations (like desaturating the |image|),
-  // then returns the |image| to the client and clears up any temporary data
-  // associated with the |request_id|.
-  void FinalizeImage(const SkBitmap* image, int request_id);
-
-  // Loads the default image for |request_id| and returns to the client.
-  void LoadDefaultImage(int request_id);
-
-  // Loads the extension's |icon| for the given |request_id| and returns the
-  // image to the client.
-  void LoadExtensionImage(const ExtensionResource& icon,
-                          int request_id);
-
-  // Loads the favicon image for the app associated with the |request_id|. If
-  // the image does not exist, we fall back to the default image.
-  void LoadFaviconImage(int request_id);
-
-  // FaviconService callback
-  void OnFaviconDataAvailable(
-      int request_id,
-      const chrome::FaviconBitmapResult& bitmap_result);
-
-  // ImageLoader callback
-  void OnImageLoaded(int request_id, const gfx::Image& image);
-
-  // Called when the extension doesn't have an icon. We fall back to multiple
-  // sources, using the following order:
-  //  1) The icons as listed in the extension / app manifests.
-  //  2) If a 16px icon and the extension has a launch URL, see if Chrome
-  //     has a corresponding favicon.
-  //  3) If still no matches, load the default extension / application icon.
-  void LoadIconFailed(int request_id);
+  // LoadExtensionImage() will load the icon using
+  // ImageLoader::LoadExtensionIconAsync(). OnIconLoaded

[Finnur, 2013/09/30 15:11:21]

OnIconLoaded... what?  :)

[dvh-g, 2013/10/01 04:19:26]

Done.

+  void OnIconLoaded(int request_id, const gfx::Image& image);
 
   // Parses and savse an ExtensionIconRequest for the URL |path| for the
   // specified |request_id|.
   bool ParseData(const std::string& path,
                  int request_id,
                  const content::URLDataSource::GotDataCallback& callback);
 
   // Stores the parameters associated with the |request_id|, making them
   // as an ExtensionIconRequest via GetData.
   void SetData(int request_id,
                const content::URLDataSource::GotDataCallback& callback,
                const Extension* extension,
                bool grayscale,
                int size,
                ExtensionIconSet::MatchType match);
 
   // Returns the ExtensionIconRequest for the given |request_id|.
   ExtensionIconRequest* GetData(int request_id);
 
   // Removes temporary data associated with |request_id|.
   void ClearData(int request_id);
 
   Profile* profile_;
 
-  // Maps tracker ids to request ids.
-  std::map<int, int> tracker_map_;
-
   // Maps request_ids to ExtensionIconRequests.
   std::map<int, ExtensionIconRequest*> request_map_;
 
-  scoped_ptr<SkBitmap> default_app_data_;
-
-  scoped_ptr<SkBitmap> default_extension_data_;
-
-  CancelableTaskTracker cancelable_task_tracker_;
-
   DISALLOW_COPY_AND_ASSIGN(ExtensionIconSource);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_UI_WEBUI_EXTENSIONS_EXTENSION_ICON_SOURCE_H_