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

chrome/browser/extensions/app_shortcut_manager.cc

Index: chrome/browser/extensions/app_shortcut_manager.cc
diff --git a/chrome/browser/extensions/app_shortcut_manager.cc b/chrome/browser/extensions/app_shortcut_manager.cc
index b6497f39d945f80a3401320cb93c88e8fca55fe5..e6f602800f88f1a8e2c4d70ae0f521b576bbd317 100644
--- a/chrome/browser/extensions/app_shortcut_manager.cc
+++ b/chrome/browser/extensions/app_shortcut_manager.cc
@@ -112,7 +112,9 @@ void AppShortcutManager::InstallApplicationShortcuts(
         size, ExtensionIconSet::MATCH_EXACTLY);
     if (!resource.empty()) {
       info_list.push_back(
-          ImageLoadingTracker::ImageInfo(resource, gfx::Size(size, size)));
+          ImageLoadingTracker::ImageInfo(resource,
+                                         gfx::Size(size, size),
+                                         ui::SCALE_FACTOR_NONE));

[pkotwicz, 2012/07/16 17:33:52]

ui::SCALE_FACTOR_100P

[xiyuan, 2012/07/16 20:11:27]

Changed but AppShortcutManager loads multi-resolution images to create an icon
out of it and does not care DIP. Shoudn't SCALE_FACTOR_NONE sounds more correct
in this sense?

     }
   }
 
@@ -130,7 +132,9 @@ void AppShortcutManager::InstallApplicationShortcuts(
           size, ExtensionIconSet::MATCH_SMALLER);
     }
     info_list.push_back(
-        ImageLoadingTracker::ImageInfo(resource, gfx::Size(size, size)));
+        ImageLoadingTracker::ImageInfo(resource,
+                                       gfx::Size(size, size),
+                                       ui::SCALE_FACTOR_NONE));

[pkotwicz, 2012/07/16 17:33:52]

Change this to ui::SCALE_FACTOR_100P

[xiyuan, 2012/07/16 20:11:27]

Done.

   }
 
   // |icon_resources| may still be empty at this point, in which case LoadImage