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 |