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

chrome/browser/extensions/extension_icon_image_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/extension_icon_image.h"
+
+#include "base/json/json_file_value_serializer.h"
+#include "base/message_loop.h"
+#include "base/path_service.h"
+#include "chrome/browser/extensions/extension_icon_image_delegate.h"
+#include "chrome/common/chrome_paths.h"
+#include "chrome/common/extensions/extension.h"
+#include "content/public/test/test_browser_thread.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using content::BrowserThread;
+using extensions::Extension;
+
+class ExtensionIconImageTest : public testing::Test,
+                               public ExtensionIconImageDelegate {
+ public:
+  ExtensionIconImageTest()
+      : image_loaded_count_(0),
+        quit_in_image_loaded_(false),
+        ui_thread_(BrowserThread::UI, &ui_loop_),
+        file_thread_(BrowserThread::FILE),
+        io_thread_(BrowserThread::IO) {
+  }
+
+  void WaitForImageLoad() {
+    quit_in_image_loaded_ = true;
+    MessageLoop::current()->Run();
+    quit_in_image_loaded_ = false;
+  }
+
+  int image_loaded_count() {
+    int result = image_loaded_count_;
+    image_loaded_count_ = 0;
+    return result;
+  }
+
+  scoped_refptr<Extension> CreateExtension(const char* name,
+                                           Extension::Location location) {
+    // Create and load an extension.
+    FilePath test_file;
+    if (!PathService::Get(chrome::DIR_TEST_DATA, &test_file)) {
+      EXPECT_FALSE(true);
+      return NULL;
+    }
+    test_file = test_file.AppendASCII("extensions")
+                         .AppendASCII(name);
+    int error_code = 0;
+    std::string error;
+    JSONFileValueSerializer serializer(test_file.AppendASCII("app.json"));
+    scoped_ptr<DictionaryValue> valid_value(
+        static_cast<DictionaryValue*>(serializer.Deserialize(&error_code,
+                                                             &error)));
+    EXPECT_EQ(0, error_code) << error;
+    if (error_code != 0)
+      return NULL;
+
+    EXPECT_TRUE(valid_value.get());
+    if (!valid_value.get())
+      return NULL;
+
+    return Extension::Create(test_file, location, *valid_value,
+                             Extension::NO_FLAGS, &error);
+  }
+
+  // testing::Test overrides:
+  virtual void SetUp() OVERRIDE {
+    file_thread_.Start();
+    io_thread_.Start();
+  }
+
+  // ExtensionIconImageDelegate overrides:
+  virtual void OnExtensionIconImageChanged(ExtensionIconImage* image) OVERRIDE {
+    image_loaded_count_++;
+    if (quit_in_image_loaded_)
+      MessageLoop::current()->Quit();
+  }
+
+ private:
+  int image_loaded_count_;
+  bool quit_in_image_loaded_;
+  MessageLoop ui_loop_;
+  content::TestBrowserThread ui_thread_;
+  content::TestBrowserThread file_thread_;
+  content::TestBrowserThread io_thread_;
+
+  DISALLOW_COPY_AND_ASSIGN(ExtensionIconImageTest);
+};
+
+TEST_F(ExtensionIconImageTest, Basic) {
+  scoped_refptr<Extension> extension(CreateExtension(
+      "image_loading_tracker", Extension::INVALID));
+  ASSERT_TRUE(extension.get() != NULL);
+
+  scoped_ptr<ExtensionIconImage> image(new ExtensionIconImage(
+      extension,
+      extension->icons(),
+      ExtensionIconSet::EXTENSION_ICON_BITTY,
+      ExtensionIconSet::MATCH_SMALLER,
+      gfx::Size(ExtensionIconSet::EXTENSION_ICON_BITTY,
+                ExtensionIconSet::EXTENSION_ICON_BITTY),
+      ImageLoadingTracker::DONT_CACHE,
+      this));
+
+  EXPECT_FALSE(image->IsEmpty());
+
+  // No representations in |image_| yet.
+  gfx::ImageSkia::ImageSkiaReps image_reps = image->image_skia().image_reps();
+  ASSERT_EQ(0u, image_reps.size());
+
+  // Gets representation for a scale factor.
+  image->image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());

[tbarzic, 2012/07/25 19:37:34]

pkotwicz 2012/07/25 02:00:39
Nit: Check the size of image_reps instead

[tbarzic, 2012/07/25 19:53:53]

the problem with image_reps is there are actually two representations added for
each new GetRepresentation (first, one null rep, and later, when the image rep
is loaded the real one); I feel like the test should not depend on that
implementation detail.

plus, there is a TODO by oshima in image_skia to make image_reps fail when
source is used (like in this case)

If you insist, I'll check image_reps size, but I'd rather keep it this way :)

+
+  gfx::ImageSkiaRep image_rep =
+      image->image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_BITTY,
+            image_rep.pixel_width());
+
+  // Gets representation for an additional scale factor.
+  image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());
+
+  image_rep = image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_SMALL,
+            image_rep.pixel_width());
+}
+
+TEST_F(ExtensionIconImageTest, BadResource) {
+  scoped_refptr<Extension> extension(CreateExtension(
+      "image_loading_tracker", Extension::INVALID));
+  ASSERT_TRUE(extension.get() != NULL);
+
+  scoped_ptr<ExtensionIconImage> image(new ExtensionIconImage(
+      extension,
+      extension->icons(),
+      ExtensionIconSet::EXTENSION_ICON_GIGANTOR,
+      ExtensionIconSet::MATCH_BIGGER,
+      gfx::Size(ExtensionIconSet::EXTENSION_ICON_GIGANTOR,
+                ExtensionIconSet::EXTENSION_ICON_GIGANTOR),
+      ImageLoadingTracker::DONT_CACHE,
+      this));
+
+  EXPECT_TRUE(image->IsEmpty());
+}
+
+TEST_F(ExtensionIconImageTest, Missing2x) {
+  scoped_refptr<Extension> extension(CreateExtension(
+      "image_loading_tracker", Extension::INVALID));
+  ASSERT_TRUE(extension.get() != NULL);
+
+  scoped_ptr<ExtensionIconImage> image(new ExtensionIconImage(
+      extension,
+      extension->icons(),
+      ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+      ExtensionIconSet::MATCH_BIGGER,
+      gfx::Size(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+                ExtensionIconSet::EXTENSION_ICON_SMALLISH),
+      ImageLoadingTracker::DONT_CACHE,
+      this));
+
+  // Gets representation for 1x.
+  image->image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());
+
+  gfx::ImageSkiaRep image_rep =
+      image->image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+            image_rep.pixel_width());
+
+  // Get representation for 2x.
+  image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+
+  image_rep = image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+
+  // 1x representation would be returned since there is no 2x resource.
+  EXPECT_EQ(ui::SCALE_FACTOR_100P, image_rep.scale_factor());
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+            image_rep.pixel_width());
+}
+
+// Similar to Missing2x test but go directly for the missing 2x resource first.
+TEST_F(ExtensionIconImageTest, FallbackTo1x) {
+  scoped_refptr<Extension> extension(CreateExtension(
+      "image_loading_tracker", Extension::INVALID));
+  ASSERT_TRUE(extension.get() != NULL);
+
+  scoped_ptr<ExtensionIconImage> image(new ExtensionIconImage(
+      extension,
+      extension->icons(),
+      ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+      ExtensionIconSet::MATCH_BIGGER,
+      gfx::Size(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+                ExtensionIconSet::EXTENSION_ICON_SMALLISH),
+      ImageLoadingTracker::DONT_CACHE,
+      this));
+
+  // Attempt to get representation for 2x.
+  image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+
+  WaitForImageLoad();
+  EXPECT_EQ(1, image_loaded_count());
+
+  // 1x representation would be returned since there is no 2x resource.
+  gfx::ImageSkiaRep image_rep =
+      image->image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
+  EXPECT_EQ(ui::SCALE_FACTOR_100P, image_rep.scale_factor());
+  EXPECT_EQ(ExtensionIconSet::EXTENSION_ICON_SMALLISH,
+            image_rep.pixel_width());
+}