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

Side by Side Diff: ui/gfx/font_list_unittest.cc

Issue 534713002: [Merge] [Win, Linux] Always display the avatar button text at the same size. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2125
Patch Set: Created 6 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 unified diff | Download patch
« no previous file with comments | « ui/gfx/font_list.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "ui/gfx/font_list.h" 5 #include "ui/gfx/font_list.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
279 FontList font_list_mix(fonts); 279 FontList font_list_mix(fonts);
280 // ascent of FontList == max(ascent of Fonts) 280 // ascent of FontList == max(ascent of Fonts)
281 EXPECT_EQ(std::max(font1.GetHeight() - font1.GetBaseline(), 281 EXPECT_EQ(std::max(font1.GetHeight() - font1.GetBaseline(),
282 font2.GetHeight() - font2.GetBaseline()), 282 font2.GetHeight() - font2.GetBaseline()),
283 font_list_mix.GetHeight() - font_list_mix.GetBaseline()); 283 font_list_mix.GetHeight() - font_list_mix.GetBaseline());
284 // descent of FontList == max(descent of Fonts) 284 // descent of FontList == max(descent of Fonts)
285 EXPECT_EQ(std::max(font1.GetBaseline(), font2.GetBaseline()), 285 EXPECT_EQ(std::max(font1.GetBaseline(), font2.GetBaseline()),
286 font_list_mix.GetBaseline()); 286 font_list_mix.GetBaseline());
287 } 287 }
288 288
289 TEST(FontListTest, Fonts_DeriveWithHeightUpperBound) {
290 std::vector<Font> fonts;
291
292 fonts.push_back(gfx::Font("Arial", 18));
293 fonts.push_back(gfx::Font("Sans serif", 18));
294 fonts.push_back(gfx::Font("Symbol", 18));
295 FontList font_list = FontList(fonts);
296
297 // A smaller upper bound should derive a font list with a smaller height.
298 const int height_1 = font_list.GetHeight() - 5;
299 FontList derived_1 = font_list.DeriveWithHeightUpperBound(height_1);
300 EXPECT_LE(derived_1.GetHeight(), height_1);
301 EXPECT_LT(derived_1.GetHeight(), font_list.GetHeight());
302 EXPECT_LT(derived_1.GetFontSize(), font_list.GetFontSize());
303
304 // A larger upper bound should not change the height of the font list.
305 const int height_2 = font_list.GetHeight() + 5;
306 FontList derived_2 = font_list.DeriveWithHeightUpperBound(height_2);
307 EXPECT_LE(derived_2.GetHeight(), height_2);
308 EXPECT_EQ(font_list.GetHeight(), derived_2.GetHeight());
309 EXPECT_EQ(font_list.GetFontSize(), derived_2.GetFontSize());
310 }
311
289 } // namespace gfx 312 } // namespace gfx
OLDNEW
« no previous file with comments | « ui/gfx/font_list.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698