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

Side by Side Diff: chrome/browser/history/select_favicon_frames_unittest.cc

Issue 335233003: Convert ui::ScaleFactor -> float in favicon/history code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix size_t Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
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
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698