| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "components/favicon_base/select_favicon_frames.h" | 5 #include "components/favicon_base/select_favicon_frames.h" |
| 6 | 6 |
| 7 #include "testing/gtest/include/gtest/gtest.h" | 7 #include "testing/gtest/include/gtest/gtest.h" |
| 8 #include "third_party/skia/include/core/SkBitmap.h" | 8 #include "third_party/skia/include/core/SkBitmap.h" |
| 9 #include "third_party/skia/include/core/SkColor.h" | 9 #include "third_party/skia/include/core/SkColor.h" |
| 10 #include "ui/base/layout.h" | |
| 11 #include "ui/gfx/image/image_skia.h" | 10 #include "ui/gfx/image/image_skia.h" |
| 12 | 11 |
| 13 using std::vector; | 12 using std::vector; |
| 14 | 13 |
| 15 namespace { | 14 namespace { |
| 16 | 15 |
| 17 vector<ui::ScaleFactor> Scale1x() { | 16 vector<float> FaviconScale1x() { |
| 18 return vector<ui::ScaleFactor>(1, ui::SCALE_FACTOR_100P); | 17 return vector<float>(1, 1.0f); |
| 19 } | 18 } |
| 20 | 19 |
| 21 vector<ui::ScaleFactor> Scale1x2x() { | 20 vector<float> FaviconScale1x2x() { |
| 22 vector<ui::ScaleFactor> scales; | 21 vector<float> scales; |
| 23 scales.push_back(ui::SCALE_FACTOR_100P); | 22 scales.push_back(1.0f); |
| 24 scales.push_back(ui::SCALE_FACTOR_200P); | 23 scales.push_back(2.0f); |
| 25 return scales; | 24 return scales; |
| 26 } | 25 } |
| 27 | 26 |
| 28 // Return gfx::Size vector with the pixel sizes of |bitmaps|. | 27 // Return gfx::Size vector with the pixel sizes of |bitmaps|. |
| 29 vector<gfx::Size> SizesFromBitmaps(const vector<SkBitmap>& bitmaps) { | 28 vector<gfx::Size> SizesFromBitmaps(const vector<SkBitmap>& bitmaps) { |
| 30 vector<gfx::Size> sizes; | 29 vector<gfx::Size> sizes; |
| 31 for (size_t i = 0; i < bitmaps.size(); ++i) | 30 for (size_t i = 0; i < bitmaps.size(); ++i) |
| 32 sizes.push_back(gfx::Size(bitmaps[i].width(), bitmaps[i].height())); | 31 sizes.push_back(gfx::Size(bitmaps[i].width(), bitmaps[i].height())); |
| 33 return sizes; | 32 return sizes; |
| 34 } | 33 } |
| 35 | 34 |
| 36 SkBitmap MakeBitmap(SkColor color, int w, int h) { | 35 SkBitmap MakeBitmap(SkColor color, int w, int h) { |
| 37 SkBitmap bitmap; | 36 SkBitmap bitmap; |
| 38 bitmap.setConfig(SkBitmap::kARGB_8888_Config, w, h); | 37 bitmap.setConfig(SkBitmap::kARGB_8888_Config, w, h); |
| 39 bitmap.allocPixels(); | 38 bitmap.allocPixels(); |
| 40 bitmap.eraseColor(color); | 39 bitmap.eraseColor(color); |
| 41 return bitmap; | 40 return bitmap; |
| 42 } | 41 } |
| 43 | 42 |
| 44 SkColor GetColor(const gfx::ImageSkia& image, ui::ScaleFactor factor, | 43 SkColor GetColor(const gfx::ImageSkia& image, float scale, |
| 45 int x = -1, int y = -1) { | 44 int x = -1, int y = -1) { |
| 46 const SkBitmap& bitmap = | 45 const SkBitmap& bitmap = |
| 47 image.GetRepresentation(ui::GetScaleForScaleFactor(factor)).sk_bitmap(); | 46 image.GetRepresentation(scale).sk_bitmap(); |
| 48 if (x == -1) | 47 if (x == -1) |
| 49 x = bitmap.width() / 2; | 48 x = bitmap.width() / 2; |
| 50 if (y == -1) | 49 if (y == -1) |
| 51 y = bitmap.width() / 2; | 50 y = bitmap.width() / 2; |
| 52 bitmap.lockPixels(); | 51 bitmap.lockPixels(); |
| 53 SkColor color = bitmap.getColor(x, y); | 52 SkColor color = bitmap.getColor(x, y); |
| 54 bitmap.unlockPixels(); | 53 bitmap.unlockPixels(); |
| 55 return color; | 54 return color; |
| 56 } | 55 } |
| 57 | 56 |
| 58 SkColor GetColor1x(const gfx::ImageSkia& image) { | 57 SkColor GetColor1x(const gfx::ImageSkia& image) { |
| 59 return GetColor(image, ui::SCALE_FACTOR_100P); | 58 return GetColor(image, 1.0f); |
| 60 } | 59 } |
| 61 | 60 |
| 62 SkColor GetColor2x(const gfx::ImageSkia& image) { | 61 SkColor GetColor2x(const gfx::ImageSkia& image) { |
| 63 return GetColor(image, ui::SCALE_FACTOR_200P); | 62 return GetColor(image, 2.0f); |
| 64 } | 63 } |
| 65 | 64 |
| 66 TEST(SelectFaviconFramesTest, ZeroSizePicksLargest) { | 65 TEST(SelectFaviconFramesTest, ZeroSizePicksLargest) { |
| 67 vector<SkBitmap> bitmaps; | 66 vector<SkBitmap> bitmaps; |
| 68 bitmaps.push_back(MakeBitmap(SK_ColorRED, 16, 16)); | 67 bitmaps.push_back(MakeBitmap(SK_ColorRED, 16, 16)); |
| 69 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 48, 48)); | 68 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 48, 48)); |
| 70 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 32, 32)); | 69 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 32, 32)); |
| 71 | 70 |
| 72 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 71 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 73 SizesFromBitmaps(bitmaps), Scale1x(), 0, NULL); | 72 SizesFromBitmaps(bitmaps), FaviconScale1x(), 0, NULL); |
| 74 EXPECT_EQ(1u, image.image_reps().size()); | 73 EXPECT_EQ(1u, image.image_reps().size()); |
| 75 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 74 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 76 EXPECT_EQ(48, image.width()); | 75 EXPECT_EQ(48, image.width()); |
| 77 EXPECT_EQ(48, image.height()); | 76 EXPECT_EQ(48, image.height()); |
| 78 | 77 |
| 79 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 78 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 80 } | 79 } |
| 81 | 80 |
| 82 TEST(SelectFaviconFramesTest, _16From16) { | 81 TEST(SelectFaviconFramesTest, _16From16) { |
| 83 vector<SkBitmap> bitmaps; | 82 vector<SkBitmap> bitmaps; |
| 84 bitmaps.push_back(MakeBitmap(SK_ColorRED, 15, 15)); | 83 bitmaps.push_back(MakeBitmap(SK_ColorRED, 15, 15)); |
| 85 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 84 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 86 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 17, 17)); | 85 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 17, 17)); |
| 87 | 86 |
| 88 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 87 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 89 SizesFromBitmaps(bitmaps), Scale1x(), 16, NULL); | 88 SizesFromBitmaps(bitmaps), FaviconScale1x(), 16, NULL); |
| 90 EXPECT_EQ(1u, image.image_reps().size()); | 89 EXPECT_EQ(1u, image.image_reps().size()); |
| 91 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 90 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 92 EXPECT_EQ(16, image.width()); | 91 EXPECT_EQ(16, image.width()); |
| 93 EXPECT_EQ(16, image.height()); | 92 EXPECT_EQ(16, image.height()); |
| 94 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 93 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 95 } | 94 } |
| 96 | 95 |
| 97 TEST(SelectFaviconFramesTest, _16From17) { | 96 TEST(SelectFaviconFramesTest, _16From17) { |
| 98 vector<SkBitmap> bitmaps; | 97 vector<SkBitmap> bitmaps; |
| 99 bitmaps.push_back(MakeBitmap(SK_ColorRED, 15, 15)); | 98 bitmaps.push_back(MakeBitmap(SK_ColorRED, 15, 15)); |
| 100 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 17, 17)); | 99 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 17, 17)); |
| 101 | 100 |
| 102 // Should resample from the bigger candidate. | 101 // Should resample from the bigger candidate. |
| 103 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 102 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 104 SizesFromBitmaps(bitmaps), Scale1x(), 16, NULL); | 103 SizesFromBitmaps(bitmaps), FaviconScale1x(), 16, NULL); |
| 105 EXPECT_EQ(1u, image.image_reps().size()); | 104 EXPECT_EQ(1u, image.image_reps().size()); |
| 106 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 105 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 107 EXPECT_EQ(16, image.width()); | 106 EXPECT_EQ(16, image.width()); |
| 108 EXPECT_EQ(16, image.height()); | 107 EXPECT_EQ(16, image.height()); |
| 109 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 108 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 110 } | 109 } |
| 111 | 110 |
| 112 TEST(SelectFaviconFramesTest, _16From15) { | 111 TEST(SelectFaviconFramesTest, _16From15) { |
| 113 vector<SkBitmap> bitmaps; | 112 vector<SkBitmap> bitmaps; |
| 114 bitmaps.push_back(MakeBitmap(SK_ColorRED, 14, 14)); | 113 bitmaps.push_back(MakeBitmap(SK_ColorRED, 14, 14)); |
| 115 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); | 114 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); |
| 116 | 115 |
| 117 // If nothing else is available, should resample from the next smaller | 116 // If nothing else is available, should resample from the next smaller |
| 118 // candidate. | 117 // candidate. |
| 119 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 118 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 120 SizesFromBitmaps(bitmaps), Scale1x(), 16, NULL); | 119 SizesFromBitmaps(bitmaps), FaviconScale1x(), 16, NULL); |
| 121 EXPECT_EQ(1u, image.image_reps().size()); | 120 EXPECT_EQ(1u, image.image_reps().size()); |
| 122 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 121 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 123 EXPECT_EQ(16, image.width()); | 122 EXPECT_EQ(16, image.width()); |
| 124 EXPECT_EQ(16, image.height()); | 123 EXPECT_EQ(16, image.height()); |
| 125 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 124 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 126 } | 125 } |
| 127 | 126 |
| 128 TEST(SelectFaviconFramesTest, _16From16_Scale2x_32_From_16) { | 127 TEST(SelectFaviconFramesTest, _16From16_Scale2x_32_From_16) { |
| 129 vector<SkBitmap> bitmaps; | 128 vector<SkBitmap> bitmaps; |
| 130 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 129 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 131 | 130 |
| 132 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 131 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 133 SizesFromBitmaps(bitmaps), Scale1x2x(), 16, NULL); | 132 SizesFromBitmaps(bitmaps), FaviconScale1x2x(), 16, NULL); |
| 134 EXPECT_EQ(2u, image.image_reps().size()); | 133 EXPECT_EQ(2u, image.image_reps().size()); |
| 135 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 134 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 136 ASSERT_TRUE(image.HasRepresentation(2.0f)); | 135 ASSERT_TRUE(image.HasRepresentation(2.0f)); |
| 137 EXPECT_EQ(16, image.width()); | 136 EXPECT_EQ(16, image.width()); |
| 138 EXPECT_EQ(16, image.height()); | 137 EXPECT_EQ(16, image.height()); |
| 139 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 138 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 140 EXPECT_EQ(SK_ColorGREEN, GetColor2x(image)); | 139 EXPECT_EQ(SK_ColorGREEN, GetColor2x(image)); |
| 141 } | 140 } |
| 142 | 141 |
| 143 TEST(SelectFaviconFramesTest, _16From16_Scale2x_32_From_32) { | 142 TEST(SelectFaviconFramesTest, _16From16_Scale2x_32_From_32) { |
| 144 vector<SkBitmap> bitmaps; | 143 vector<SkBitmap> bitmaps; |
| 145 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 144 bitmaps.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 146 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 32, 32)); | 145 bitmaps.push_back(MakeBitmap(SK_ColorBLUE, 32, 32)); |
| 147 | 146 |
| 148 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, | 147 gfx::ImageSkia image = SelectFaviconFrames(bitmaps, |
| 149 SizesFromBitmaps(bitmaps), Scale1x2x(), 16, NULL); | 148 SizesFromBitmaps(bitmaps), FaviconScale1x2x(), 16, NULL); |
| 150 EXPECT_EQ(2u, image.image_reps().size()); | 149 EXPECT_EQ(2u, image.image_reps().size()); |
| 151 ASSERT_TRUE(image.HasRepresentation(1.0f)); | 150 ASSERT_TRUE(image.HasRepresentation(1.0f)); |
| 152 ASSERT_TRUE(image.HasRepresentation(2.0f)); | 151 ASSERT_TRUE(image.HasRepresentation(2.0f)); |
| 153 EXPECT_EQ(16, image.width()); | 152 EXPECT_EQ(16, image.width()); |
| 154 EXPECT_EQ(16, image.height()); | 153 EXPECT_EQ(16, image.height()); |
| 155 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); | 154 EXPECT_EQ(SK_ColorGREEN, GetColor1x(image)); |
| 156 EXPECT_EQ(SK_ColorBLUE, GetColor2x(image)); | 155 EXPECT_EQ(SK_ColorBLUE, GetColor2x(image)); |
| 157 } | 156 } |
| 158 | 157 |
| 159 TEST(SelectFaviconFramesTest, ExactMatchBetterThanLargeBitmap) { | 158 TEST(SelectFaviconFramesTest, ExactMatchBetterThanLargeBitmap) { |
| 160 float score1; | 159 float score1; |
| 161 vector<SkBitmap> bitmaps1; | 160 vector<SkBitmap> bitmaps1; |
| 162 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 48, 48)); | 161 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 48, 48)); |
| 163 SelectFaviconFrames(bitmaps1, | 162 SelectFaviconFrames(bitmaps1, |
| 164 SizesFromBitmaps(bitmaps1), Scale1x2x(), 16, &score1); | 163 SizesFromBitmaps(bitmaps1), FaviconScale1x2x(), 16, &score1); |
| 165 | 164 |
| 166 float score2; | 165 float score2; |
| 167 vector<SkBitmap> bitmaps2; | 166 vector<SkBitmap> bitmaps2; |
| 168 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 167 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 169 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 32, 32)); | 168 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 32, 32)); |
| 170 SelectFaviconFrames(bitmaps2, | 169 SelectFaviconFrames(bitmaps2, |
| 171 SizesFromBitmaps(bitmaps2), Scale1x2x(), 16, &score2); | 170 SizesFromBitmaps(bitmaps2), FaviconScale1x2x(), 16, &score2); |
| 172 | 171 |
| 173 EXPECT_GT(score2, score1); | 172 EXPECT_GT(score2, score1); |
| 174 } | 173 } |
| 175 | 174 |
| 176 TEST(SelectFaviconFramesTest, UpsampleABitBetterThanHugeBitmap) { | 175 TEST(SelectFaviconFramesTest, UpsampleABitBetterThanHugeBitmap) { |
| 177 float score1; | 176 float score1; |
| 178 vector<SkBitmap> bitmaps1; | 177 vector<SkBitmap> bitmaps1; |
| 179 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 128, 128)); | 178 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 128, 128)); |
| 180 SelectFaviconFrames(bitmaps1, | 179 SelectFaviconFrames(bitmaps1, |
| 181 SizesFromBitmaps(bitmaps1), Scale1x2x(), 16, &score1); | 180 SizesFromBitmaps(bitmaps1), FaviconScale1x2x(), 16, &score1); |
| 182 | 181 |
| 183 float score2; | 182 float score2; |
| 184 vector<SkBitmap> bitmaps2; | 183 vector<SkBitmap> bitmaps2; |
| 185 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 24, 24)); | 184 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 24, 24)); |
| 186 SelectFaviconFrames(bitmaps2, | 185 SelectFaviconFrames(bitmaps2, |
| 187 SizesFromBitmaps(bitmaps2), Scale1x2x(), 16, &score2); | 186 SizesFromBitmaps(bitmaps2), FaviconScale1x2x(), 16, &score2); |
| 188 | 187 |
| 189 float score3; | 188 float score3; |
| 190 vector<SkBitmap> bitmaps3; | 189 vector<SkBitmap> bitmaps3; |
| 191 bitmaps3.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 190 bitmaps3.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 192 SelectFaviconFrames(bitmaps3, | 191 SelectFaviconFrames(bitmaps3, |
| 193 SizesFromBitmaps(bitmaps3), Scale1x2x(), 16, &score3); | 192 SizesFromBitmaps(bitmaps3), FaviconScale1x2x(), 16, &score3); |
| 194 | 193 |
| 195 float score4; | 194 float score4; |
| 196 vector<SkBitmap> bitmaps4; | 195 vector<SkBitmap> bitmaps4; |
| 197 bitmaps4.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); | 196 bitmaps4.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); |
| 198 SelectFaviconFrames(bitmaps4, | 197 SelectFaviconFrames(bitmaps4, |
| 199 SizesFromBitmaps(bitmaps4), Scale1x2x(), 16, &score4); | 198 SizesFromBitmaps(bitmaps4), FaviconScale1x2x(), 16, &score4); |
| 200 | 199 |
| 201 EXPECT_GT(score2, score1); | 200 EXPECT_GT(score2, score1); |
| 202 EXPECT_GT(score3, score1); | 201 EXPECT_GT(score3, score1); |
| 203 EXPECT_GT(score4, score1); | 202 EXPECT_GT(score4, score1); |
| 204 } | 203 } |
| 205 | 204 |
| 206 TEST(SelectFaviconFramesTest, DownsamplingBetterThanUpsampling) { | 205 TEST(SelectFaviconFramesTest, DownsamplingBetterThanUpsampling) { |
| 207 float score1; | 206 float score1; |
| 208 vector<SkBitmap> bitmaps1; | 207 vector<SkBitmap> bitmaps1; |
| 209 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 8, 8)); | 208 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 8, 8)); |
| 210 SelectFaviconFrames(bitmaps1, | 209 SelectFaviconFrames(bitmaps1, |
| 211 SizesFromBitmaps(bitmaps1), Scale1x(), 16, &score1); | 210 SizesFromBitmaps(bitmaps1), FaviconScale1x(), 16, &score1); |
| 212 | 211 |
| 213 float score2; | 212 float score2; |
| 214 vector<SkBitmap> bitmaps2; | 213 vector<SkBitmap> bitmaps2; |
| 215 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 24, 24)); | 214 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 24, 24)); |
| 216 SelectFaviconFrames(bitmaps2, | 215 SelectFaviconFrames(bitmaps2, |
| 217 SizesFromBitmaps(bitmaps2), Scale1x(), 16, &score2); | 216 SizesFromBitmaps(bitmaps2), FaviconScale1x(), 16, &score2); |
| 218 | 217 |
| 219 EXPECT_GT(score2, score1); | 218 EXPECT_GT(score2, score1); |
| 220 } | 219 } |
| 221 | 220 |
| 222 TEST(SelectFaviconFramesTest, DownsamplingLessIsBetter) { | 221 TEST(SelectFaviconFramesTest, DownsamplingLessIsBetter) { |
| 223 float score1; | 222 float score1; |
| 224 vector<SkBitmap> bitmaps1; | 223 vector<SkBitmap> bitmaps1; |
| 225 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 34, 34)); | 224 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 34, 34)); |
| 226 SelectFaviconFrames(bitmaps1, | 225 SelectFaviconFrames(bitmaps1, |
| 227 SizesFromBitmaps(bitmaps1), Scale1x2x(), 16, &score1); | 226 SizesFromBitmaps(bitmaps1), FaviconScale1x2x(), 16, &score1); |
| 228 | 227 |
| 229 float score2; | 228 float score2; |
| 230 vector<SkBitmap> bitmaps2; | 229 vector<SkBitmap> bitmaps2; |
| 231 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 33, 33)); | 230 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 33, 33)); |
| 232 SelectFaviconFrames(bitmaps2, | 231 SelectFaviconFrames(bitmaps2, |
| 233 SizesFromBitmaps(bitmaps2), Scale1x2x(), 16, &score2); | 232 SizesFromBitmaps(bitmaps2), FaviconScale1x2x(), 16, &score2); |
| 234 | 233 |
| 235 EXPECT_GT(score2, score1); | 234 EXPECT_GT(score2, score1); |
| 236 } | 235 } |
| 237 | 236 |
| 238 TEST(SelectFaviconFramesTest, UpsamplingLessIsBetter) { | 237 TEST(SelectFaviconFramesTest, UpsamplingLessIsBetter) { |
| 239 float score1; | 238 float score1; |
| 240 vector<SkBitmap> bitmaps1; | 239 vector<SkBitmap> bitmaps1; |
| 241 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 8, 8)); | 240 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 8, 8)); |
| 242 SelectFaviconFrames(bitmaps1, | 241 SelectFaviconFrames(bitmaps1, |
| 243 SizesFromBitmaps(bitmaps1), Scale1x2x(), 16, &score1); | 242 SizesFromBitmaps(bitmaps1), FaviconScale1x2x(), 16, &score1); |
| 244 | 243 |
| 245 float score2; | 244 float score2; |
| 246 vector<SkBitmap> bitmaps2; | 245 vector<SkBitmap> bitmaps2; |
| 247 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 9, 9)); | 246 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 9, 9)); |
| 248 SelectFaviconFrames(bitmaps2, | 247 SelectFaviconFrames(bitmaps2, |
| 249 SizesFromBitmaps(bitmaps2), Scale1x2x(), 16, &score2); | 248 SizesFromBitmaps(bitmaps2), FaviconScale1x2x(), 16, &score2); |
| 250 | 249 |
| 251 EXPECT_GT(score2, score1); | 250 EXPECT_GT(score2, score1); |
| 252 } | 251 } |
| 253 | 252 |
| 254 // Test that the score is determined by the |original_sizes| parameter, not the | 253 // Test that the score is determined by the |original_sizes| parameter, not the |
| 255 // |bitmaps| parameter to SelectFaviconFrames(). | 254 // |bitmaps| parameter to SelectFaviconFrames(). |
| 256 TEST(SelectFaviconFramesTest, ScoreDeterminedByOriginalSizes) { | 255 TEST(SelectFaviconFramesTest, ScoreDeterminedByOriginalSizes) { |
| 257 vector<SkBitmap> bitmaps1; | 256 vector<SkBitmap> bitmaps1; |
| 258 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); | 257 bitmaps1.push_back(MakeBitmap(SK_ColorGREEN, 16, 16)); |
| 259 vector<gfx::Size> sizes1; | 258 vector<gfx::Size> sizes1; |
| 260 sizes1.push_back(gfx::Size(256, 256)); | 259 sizes1.push_back(gfx::Size(256, 256)); |
| 261 float score1; | 260 float score1; |
| 262 SelectFaviconFrames(bitmaps1, sizes1, Scale1x(), 16, &score1); | 261 SelectFaviconFrames(bitmaps1, sizes1, FaviconScale1x(), 16, &score1); |
| 263 | 262 |
| 264 vector<SkBitmap> bitmaps2; | 263 vector<SkBitmap> bitmaps2; |
| 265 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); | 264 bitmaps2.push_back(MakeBitmap(SK_ColorGREEN, 15, 15)); |
| 266 vector<gfx::Size> sizes2; | 265 vector<gfx::Size> sizes2; |
| 267 sizes2.push_back(gfx::Size(15, 15)); | 266 sizes2.push_back(gfx::Size(15, 15)); |
| 268 float score2; | 267 float score2; |
| 269 SelectFaviconFrames(bitmaps2, sizes2, Scale1x(), 16, &score2); | 268 SelectFaviconFrames(bitmaps2, sizes2, FaviconScale1x(), 16, &score2); |
| 270 | 269 |
| 271 EXPECT_GT(score2, score1); | 270 EXPECT_GT(score2, score1); |
| 272 } | 271 } |
| 273 | 272 |
| 274 } // namespace | 273 } // namespace |
| OLD | NEW |