Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(606)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: trunk/src/chrome/browser/extensions/extension_icon_image_unittest.cc

Issue 24262008: Revert 224473 "Remove dependency on ui::ScaleFactor from ui/gfx" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « trunk/src/chrome/browser/extensions/extension_icon_image.cc ('k') | trunk/src/chrome/browser/extensions/extension_icon_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/chrome/browser/extensions/extension_icon_image_unittest.cc
===================================================================
--- trunk/src/chrome/browser/extensions/extension_icon_image_unittest.cc (revision 224497)
+++ trunk/src/chrome/browser/extensions/extension_icon_image_unittest.cc (working copy)
@@ -31,7 +31,7 @@
SkBitmap CreateBlankBitmapForScale(int size_dip, ui::ScaleFactor scale_factor) {
SkBitmap bitmap;
- const float scale = ui::GetImageScale(scale_factor);
+ const float scale = ui::GetScaleFactorScale(scale_factor);
bitmap.setConfig(SkBitmap::kARGB_8888_Config,
static_cast<int>(size_dip * scale),
static_cast<int>(size_dip * scale));
@@ -59,8 +59,9 @@
}
virtual ~MockImageSkiaSource() {}
- virtual gfx::ImageSkiaRep GetImageForScale(float scale) OVERRIDE {
- return image_.GetRepresentation(scale);
+ virtual gfx::ImageSkiaRep GetImageForScale(
+ ui::ScaleFactor scale_factor) OVERRIDE {
+ return image_.GetRepresentation(scale_factor);
}
private:
@@ -209,10 +210,6 @@
} // namespace
TEST_F(ExtensionIconImageTest, Basic) {
- std::vector<ui::ScaleFactor> supported_factors;
- supported_factors.push_back(ui::SCALE_FACTOR_100P);
- supported_factors.push_back(ui::SCALE_FACTOR_200P);
- ui::test::ScopedSetSupportedScaleFactors scoped_supported(supported_factors);
scoped_ptr<Profile> profile(new TestingProfile());
scoped_refptr<Extension> extension(CreateExtension(
"extension_icon_image", Manifest::INVALID_LOCATION));
@@ -243,7 +240,8 @@
ASSERT_EQ(0u, image_reps.size());
// Gets representation for a scale factor.
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
// Before the image representation is loaded, image should contain blank
// image representation.
@@ -255,14 +253,14 @@
EXPECT_EQ(1, ImageLoadedCount());
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(1.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
// We should get the right representation now.
EXPECT_TRUE(gfx::BitmapsAreEqual(representation.sk_bitmap(), bitmap_16));
EXPECT_EQ(16, representation.pixel_width());
// Gets representation for an additional scale factor.
- representation = image.image_skia().GetRepresentation(2.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
@@ -272,7 +270,7 @@
EXPECT_EQ(1, ImageLoadedCount());
ASSERT_EQ(2u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(2.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
// Image should have been resized.
EXPECT_EQ(32, representation.pixel_width());
@@ -283,10 +281,6 @@
// There is no resource with either exact or bigger size, but there is a smaller
// resource.
TEST_F(ExtensionIconImageTest, FallbackToSmallerWhenNoBigger) {
- std::vector<ui::ScaleFactor> supported_factors;
- supported_factors.push_back(ui::SCALE_FACTOR_100P);
- supported_factors.push_back(ui::SCALE_FACTOR_200P);
- ui::test::ScopedSetSupportedScaleFactors scoped_supported(supported_factors);
scoped_ptr<Profile> profile(new TestingProfile());
scoped_refptr<Extension> extension(CreateExtension(
"extension_icon_image", Manifest::INVALID_LOCATION));
@@ -306,17 +300,18 @@
default_icon,
this);
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(2.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
WaitForImageLoad();
EXPECT_EQ(1, ImageLoadedCount());
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(2.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_200P);
// We should have loaded the biggest smaller resource resized to the actual
// size.
- EXPECT_EQ(2.0f, representation.scale());
+ EXPECT_EQ(ui::SCALE_FACTOR_200P, representation.scale_factor());
EXPECT_EQ(64, representation.pixel_width());
EXPECT_TRUE(gfx::BitmapsAreEqual(representation.sk_bitmap(),
EnsureBitmapSize(bitmap_48, 64)));
@@ -345,16 +340,17 @@
default_icon,
this);
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
WaitForImageLoad();
EXPECT_EQ(1, ImageLoadedCount());
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(1.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
// We should have loaded smaller (resized) resource.
- EXPECT_EQ(1.0f, representation.scale());
+ EXPECT_EQ(ui::SCALE_FACTOR_100P, representation.scale_factor());
EXPECT_EQ(17, representation.pixel_width());
EXPECT_TRUE(gfx::BitmapsAreEqual(representation.sk_bitmap(),
EnsureBitmapSize(bitmap_16, 17)));
@@ -379,22 +375,23 @@
default_icon,
this);
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
EnsureBitmapSize(
- default_icon.GetRepresentation(1.0f).sk_bitmap(),
+ default_icon.GetRepresentation(ui::SCALE_FACTOR_100P).sk_bitmap(),
kRequestedSize)));
EXPECT_EQ(0, ImageLoadedCount());
// We should have a default icon representation.
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(1.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
EnsureBitmapSize(
- default_icon.GetRepresentation(1.0f).sk_bitmap(),
+ default_icon.GetRepresentation(ui::SCALE_FACTOR_100P).sk_bitmap(),
kRequestedSize)));
}
@@ -420,7 +417,8 @@
default_icon,
this);
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
CreateBlankBitmapForScale(kInvalidIconSize, ui::SCALE_FACTOR_100P)));
@@ -430,11 +428,11 @@
// We should have default icon representation now.
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- representation = image.image_skia().GetRepresentation(1.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
EnsureBitmapSize(
- default_icon.GetRepresentation(1.0f).sk_bitmap(),
+ default_icon.GetRepresentation(ui::SCALE_FACTOR_100P).sk_bitmap(),
kInvalidIconSize)));
}
@@ -460,16 +458,17 @@
lazy_default_icon,
this);
- ASSERT_FALSE(lazy_default_icon.HasRepresentation(1.0f));
+ ASSERT_FALSE(lazy_default_icon.HasRepresentation(ui::SCALE_FACTOR_100P));
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
// The resouce set is empty, so we should get the result right away.
- EXPECT_TRUE(lazy_default_icon.HasRepresentation(1.0f));
+ EXPECT_TRUE(lazy_default_icon.HasRepresentation(ui::SCALE_FACTOR_100P));
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
EnsureBitmapSize(
- default_icon.GetRepresentation(1.0f).sk_bitmap(),
+ default_icon.GetRepresentation(ui::SCALE_FACTOR_100P).sk_bitmap(),
kRequestedSize)));
// We should have a default icon representation.
@@ -499,22 +498,23 @@
lazy_default_icon,
this);
- ASSERT_FALSE(lazy_default_icon.HasRepresentation(1.0f));
+ ASSERT_FALSE(lazy_default_icon.HasRepresentation(ui::SCALE_FACTOR_100P));
- gfx::ImageSkiaRep representation = image.image_skia().GetRepresentation(1.0f);
+ gfx::ImageSkiaRep representation =
+ image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
WaitForImageLoad();
EXPECT_EQ(1, ImageLoadedCount());
// We should have default icon representation now.
ASSERT_EQ(1u, image.image_skia().image_reps().size());
- EXPECT_TRUE(lazy_default_icon.HasRepresentation(1.0f));
+ EXPECT_TRUE(lazy_default_icon.HasRepresentation(ui::SCALE_FACTOR_100P));
- representation = image.image_skia().GetRepresentation(1.0f);
+ representation = image.image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
EnsureBitmapSize(
- default_icon.GetRepresentation(1.0f).sk_bitmap(),
+ default_icon.GetRepresentation(ui::SCALE_FACTOR_100P).sk_bitmap(),
kInvalidIconSize)));
}
@@ -545,7 +545,7 @@
// Load an image representation.
gfx::ImageSkiaRep representation =
- image->image_skia().GetRepresentation(1.0f);
+ image->image_skia().GetRepresentation(ui::SCALE_FACTOR_100P);
WaitForImageLoad();
EXPECT_EQ(1, ImageLoadedCount());
@@ -557,14 +557,14 @@
extension = NULL;
// Image skia should still be able to get previously loaded representation.
- representation = image_skia.GetRepresentation(1.0f);
+ representation = image_skia.GetRepresentation(ui::SCALE_FACTOR_100P);
- EXPECT_EQ(1.0f, representation.scale());
+ EXPECT_EQ(ui::SCALE_FACTOR_100P, representation.scale_factor());
EXPECT_EQ(16, representation.pixel_width());
EXPECT_TRUE(gfx::BitmapsAreEqual(representation.sk_bitmap(), bitmap_16));
// When requesting another representation, we should get blank image.
- representation = image_skia.GetRepresentation(2.0f);
+ representation = image_skia.GetRepresentation(ui::SCALE_FACTOR_200P);
EXPECT_TRUE(gfx::BitmapsAreEqual(
representation.sk_bitmap(),
« no previous file with comments | « trunk/src/chrome/browser/extensions/extension_icon_image.cc ('k') | trunk/src/chrome/browser/extensions/extension_icon_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698