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

Unified Diff: ui/gfx/platform_font_win_unittest.cc

Issue 854713003: More old files deletion. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fix tryjobs? Created 5 years, 11 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 | « ui/gfx/platform_font_win.cc ('k') | ui/gfx/range/OWNERS » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/platform_font_win_unittest.cc
diff --git a/ui/gfx/platform_font_win_unittest.cc b/ui/gfx/platform_font_win_unittest.cc
deleted file mode 100644
index 08e5d0840a58965faeed4a2b43bbd7b1e1b858df..0000000000000000000000000000000000000000
--- a/ui/gfx/platform_font_win_unittest.cc
+++ /dev/null
@@ -1,120 +0,0 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "ui/gfx/platform_font_win.h"
-
-#include "base/logging.h"
-#include "base/memory/ref_counted.h"
-#include "testing/gtest/include/gtest/gtest.h"
-#include "ui/gfx/font.h"
-
-namespace gfx {
-
-namespace {
-
-// Returns a font based on |base_font| with height at most |target_height| and
-// font size maximized. Returns |base_font| if height is already equal.
-gfx::Font AdjustFontSizeForHeight(const gfx::Font& base_font,
- int target_height) {
- Font expected_font = base_font;
- if (base_font.GetHeight() < target_height) {
- // Increase size while height is <= |target_height|.
- Font larger_font = base_font.Derive(1, 0);
- while (larger_font.GetHeight() <= target_height) {
- expected_font = larger_font;
- larger_font = larger_font.Derive(1, 0);
- }
- } else if (expected_font.GetHeight() > target_height) {
- // Decrease size until height is <= |target_height|.
- do {
- expected_font = expected_font.Derive(-1, 0);
- } while (expected_font.GetHeight() > target_height);
- }
- return expected_font;
-}
-
-} // namespace
-
-TEST(PlatformFontWinTest, DeriveFontWithHeight) {
- const Font base_font;
- PlatformFontWin* platform_font =
- static_cast<PlatformFontWin*>(base_font.platform_font());
-
- for (int i = -10; i < 10; i++) {
- const int target_height = base_font.GetHeight() + i;
- Font expected_font = AdjustFontSizeForHeight(base_font, target_height);
- ASSERT_LE(expected_font.GetHeight(), target_height);
-
- Font derived_font = platform_font->DeriveFontWithHeight(target_height, 0);
- EXPECT_EQ(expected_font.GetFontName(), derived_font.GetFontName());
- EXPECT_EQ(expected_font.GetFontSize(), derived_font.GetFontSize());
- EXPECT_LE(expected_font.GetHeight(), target_height);
- EXPECT_EQ(0, derived_font.GetStyle());
-
- derived_font = platform_font->DeriveFontWithHeight(target_height,
- Font::BOLD);
- EXPECT_EQ(expected_font.GetFontName(), derived_font.GetFontName());
- EXPECT_EQ(expected_font.GetFontSize(), derived_font.GetFontSize());
- EXPECT_LE(expected_font.GetHeight(), target_height);
- EXPECT_EQ(Font::BOLD, derived_font.GetStyle());
-
- // Test that deriving from the new font has the expected result.
- Font rederived_font = derived_font.Derive(1, 0);
- expected_font = Font(derived_font.GetFontName(),
- derived_font.GetFontSize() + 1);
- EXPECT_EQ(expected_font.GetFontName(), rederived_font.GetFontName());
- EXPECT_EQ(expected_font.GetFontSize(), rederived_font.GetFontSize());
- EXPECT_EQ(expected_font.GetHeight(), rederived_font.GetHeight());
- }
-}
-
-TEST(PlatformFontWinTest, DeriveFontWithHeight_Consistency) {
- gfx::Font arial_12("Arial", 12);
- ASSERT_GT(16, arial_12.GetHeight());
- gfx::Font derived_1 = static_cast<PlatformFontWin*>(
- arial_12.platform_font())->DeriveFontWithHeight(16, 0);
-
- gfx::Font arial_15("Arial", 15);
- ASSERT_LT(16, arial_15.GetHeight());
- gfx::Font derived_2 = static_cast<PlatformFontWin*>(
- arial_15.platform_font())->DeriveFontWithHeight(16, 0);
-
- EXPECT_EQ(derived_1.GetFontSize(), derived_2.GetFontSize());
- EXPECT_EQ(16, derived_1.GetHeight());
- EXPECT_EQ(16, derived_2.GetHeight());
-}
-
-// Callback function used by DeriveFontWithHeight_MinSize() below.
-static int GetMinFontSize() {
- return 10;
-}
-
-TEST(PlatformFontWinTest, DeriveFontWithHeight_MinSize) {
- PlatformFontWin::GetMinimumFontSizeCallback old_callback =
- PlatformFontWin::get_minimum_font_size_callback;
- PlatformFontWin::get_minimum_font_size_callback = &GetMinFontSize;
-
- const Font base_font;
- const Font min_font(base_font.GetFontName(), GetMinFontSize());
- PlatformFontWin* platform_font =
- static_cast<PlatformFontWin*>(base_font.platform_font());
-
- const Font derived_font =
- platform_font->DeriveFontWithHeight(min_font.GetHeight() - 1, 0);
- EXPECT_EQ(min_font.GetFontSize(), derived_font.GetFontSize());
- EXPECT_EQ(min_font.GetHeight(), derived_font.GetHeight());
-
- PlatformFontWin::get_minimum_font_size_callback = old_callback;
-}
-
-TEST(PlatformFontWinTest, DeriveFontWithHeight_TooSmall) {
- const Font base_font;
- PlatformFontWin* platform_font =
- static_cast<PlatformFontWin*>(base_font.platform_font());
-
- const Font derived_font = platform_font->DeriveFontWithHeight(1, 0);
- EXPECT_GT(derived_font.GetHeight(), 1);
-}
-
-} // namespace gfx
« no previous file with comments | « ui/gfx/platform_font_win.cc ('k') | ui/gfx/range/OWNERS » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698