chromeos: Fix pixelated icons in app list and launcher (part 3)

chrome/browser/ui/ash/app_list/extension_app_item.cc

 // 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.
 
 #include "chrome/browser/ui/ash/app_list/extension_app_item.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_prefs.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
@@ skipping 120 matching lines @@
 ExtensionAppItem::~ExtensionAppItem() {
 }
 
 const Extension* ExtensionAppItem::GetExtension() const {
   const Extension* extension =
     profile_->GetExtensionService()->GetInstalledExtension(extension_id_);
   return extension;
 }
 
 void ExtensionAppItem::LoadImage(const Extension* extension) {
-  tracker_.reset(new ImageLoadingTracker(this));
-  tracker_->LoadImage(extension,
-                      extension->GetIconResource(
-                          extension_misc::EXTENSION_ICON_LARGE,
-                          ExtensionIconSet::MATCH_BIGGER),
-                      gfx::Size(extension_misc::EXTENSION_ICON_LARGE,
-                                extension_misc::EXTENSION_ICON_LARGE),
-                      ImageLoadingTracker::DONT_CACHE);
+  icon_.reset(new extensions::IconImage(
+      extension,
+      extension->icons(),
+      extension_misc::EXTENSION_ICON_MEDIUM,
+      this));
+  OnExtensionIconImageChanged(icon_.get());
 }
 
 void ExtensionAppItem::ShowExtensionOptions() {
   const Extension* extension = GetExtension();
   if (!extension)
     return;
 
   chrome::NavigateParams params(profile_,
                                 extension->options_url(),
                                 content::PAGE_TRANSITION_LINK);
   chrome::Navigate(&params);
 }
 
 void ExtensionAppItem::StartExtensionUninstall() {
   // ExtensionUninstall deletes itself when done or aborted.
   ExtensionUninstaller* uninstaller = new ExtensionUninstaller(profile_,
                                                                extension_id_);
   uninstaller->Run();
 }
 
-void ExtensionAppItem::OnImageLoaded(const gfx::Image& image,
-                                     const std::string& extension_id,
-                                     int tracker_index) {
-  if (!image.IsEmpty())
-    SetIcon(*image.ToImageSkia());
-  else
-    SetIcon(Extension::GetDefaultIcon(true /* is_app */));
+void ExtensionAppItem::OnExtensionIconImageChanged(
+    extensions::IconImage* image) {
+  DCHECK(icon_.get() == image);
+  SetIcon(image->image_skia());
+}
+
+void ExtensionAppItem::OnIconImageLoadFailed(
+    extensions::IconImage* image,
+    ui::ScaleFactor scale_factor) {
+  DCHECK(icon_.get() == image);
+
+  image->SetRepresentation(gfx::ImageSkiaRep(
+      Extension::GetDefaultIcon(true), scale_factor));
 }
 
 bool ExtensionAppItem::IsItemForCommandIdDynamic(int command_id) const {
   return command_id == TOGGLE_PIN;
 }
 
 string16 ExtensionAppItem::GetLabelForCommandId(int command_id) const {
   if (command_id == TOGGLE_PIN) {
     return controller_->IsAppPinned(extension_id_) ?
         l10n_util::GetStringUTF16(IDS_APP_LIST_CONTEXT_MENU_UNPIN) :
@@ skipping 94 matching lines @@
         LAUNCH_TYPE_FULLSCREEN,
         IDS_APP_CONTEXT_MENU_OPEN_MAXIMIZED);
     context_menu_model_->AddSeparator();
     context_menu_model_->AddItemWithStringId(OPTIONS, IDS_NEW_TAB_APP_OPTIONS);
     context_menu_model_->AddItemWithStringId(UNINSTALL,
                                              IDS_EXTENSIONS_UNINSTALL);
   }
 
   return context_menu_model_.get();
 }