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

chrome/browser/extensions/image_loading_tracker_unittest.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/extensions/image_loading_tracker.h"
+
+#include <algorithm>
+
 #include "base/json/json_file_value_serializer.h"
 #include "base/message_loop.h"
 #include "base/path_service.h"
-#include "chrome/browser/extensions/image_loading_tracker.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/extensions/extension_icon_set.h"
 #include "chrome/common/extensions/extension_resource.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/test/test_browser_thread.h"
 #include "grit/component_extension_resources.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/skia/include/core/SkBitmap.h"
@@ skipping 178 matching lines @@
       "image_loading_tracker", Extension::INVALID));
   ASSERT_TRUE(extension.get() != NULL);
 
   std::vector<ImageLoadingTracker::ImageInfo> info_list;
   int sizes[] = {ExtensionIconSet::EXTENSION_ICON_SMALLISH,
                  ExtensionIconSet::EXTENSION_ICON_BITTY};
   for (size_t i = 0; i < arraysize(sizes); ++i) {
     ExtensionResource resource =
         extension->GetIconResource(sizes[i], ExtensionIconSet::MATCH_EXACTLY);
     info_list.push_back(ImageLoadingTracker::ImageInfo(
-        resource, gfx::Size(sizes[i], sizes[i])));
+        resource, gfx::Size(sizes[i], sizes[i]), ui::SCALE_FACTOR_NONE));
   }
 
   ImageLoadingTracker loader(this);
   loader.LoadImages(extension.get(), info_list, ImageLoadingTracker::CACHE);
 
   // The image isn't cached, so we should not have received notification.
   EXPECT_EQ(0, image_loaded_count());
 
   WaitForImageLoad();
 
@@ skipping 28 matching lines @@
 #if defined(FILE_MANAGER_EXTENSION)
   ImageLoadingTracker loader(this);
   int resource_id;
   ASSERT_EQ(true,
             loader.IsComponentExtensionResource(extension.get(),
                                                 resource,
                                                 resource_id));
   ASSERT_EQ(IDR_FILE_MANAGER_ICON_16, resource_id);
 #endif
 }
+
+// Tests loading image that supports DIP.
+TEST_F(ImageLoadingTrackerTest, LoadImageInDIP) {
+  scoped_refptr<Extension> extension(CreateExtension(
+      "image_loading_tracker", Extension::INVALID));
+  ASSERT_TRUE(extension.get() != NULL);
+
+  ImageLoadingTracker loader(this);
+  loader.LoadImageInDIP(extension,
+                        ExtensionIconSet::EXTENSION_ICON_BITTY,
+                        ExtensionIconSet::MATCH_SMALLER,
+                        gfx::Size(ExtensionIconSet::EXTENSION_ICON_BITTY,
+                                  ExtensionIconSet::EXTENSION_ICON_BITTY),
+                        ImageLoadingTracker::DONT_CACHE);
+
+  EXPECT_EQ(0, image_loaded_count());
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());
+
+  std::vector<gfx::ImageSkiaRep> image_reps =
+      image_.ToImageSkia()->image_reps();
+  ASSERT_EQ(1u, image_reps.size());
+  EXPECT_EQ(ui::SCALE_FACTOR_200P, image_reps[0].scale_factor());
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+            image_reps[0].pixel_width());
+
+  // Nows, asks image for a different scale factor.
+  image_.ToImageSkia()->GetRepresentation(ui::SCALE_FACTOR_100P);
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());
+
+  image_reps = image_.ToImageSkia()->image_reps();
+  // TODO(xiyaun): Uncomment the following after peter's CL that keeps
+  // only one representation for one scale factor change is in.
+  // ASSERT_EQ(2u, image_reps.size());
+  gfx::ImageSkiaRep image_rep =
+      image_.ToImageSkia()->GetRepresentation(ui::SCALE_FACTOR_100P);
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_BITTY,
+            image_rep.pixel_width());
+}
+