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

Unified Diff: ash/display/display_manager_unittest.cc

Issue 2945913003: Add 1.6x mode. (Closed)
Patch Set: Add 1.6x support Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | ui/display/manager/chromeos/display_change_observer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/display/display_manager_unittest.cc
diff --git a/ash/display/display_manager_unittest.cc b/ash/display/display_manager_unittest.cc
index 924f2148278ae1bfc63eb93cb8839471249b2212..fb07dc0e1bc97946762dd0992e302ddc81650e5e 100644
--- a/ash/display/display_manager_unittest.cc
+++ b/ash/display/display_manager_unittest.cc
@@ -1673,8 +1673,6 @@ TEST_F(DisplayManagerTest, Rotate) {
}
TEST_F(DisplayManagerTest, UIScale) {
- display::test::ScopedDisable125DSFForUIScaling disable;
-
UpdateDisplay("1280x800");
int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
display::test::DisplayManagerTestApi(display_manager())
@@ -1785,17 +1783,6 @@ TEST_F(DisplayManagerTest, UIScale) {
display = display::Screen::GetScreen()->GetPrimaryDisplay();
EXPECT_EQ(1.0f, display.device_scale_factor());
EXPECT_EQ("1280x850", display.bounds().size().ToString());
-
- // 1.25 ui scaling on 1.25 DSF device should use 1.0 DSF
- // on screen.
- UpdateDisplay("1280x850*1.25");
- display::test::DisplayManagerTestApi(display_manager())
- .SetDisplayUIScale(display_id, 1.25f);
- EXPECT_EQ(1.25f, GetDisplayInfoAt(0).configured_ui_scale());
- EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
- display = display::Screen::GetScreen()->GetPrimaryDisplay();
- EXPECT_EQ(1.0f, display.device_scale_factor());
- EXPECT_EQ("1280x850", display.bounds().size().ToString());
}
TEST_F(DisplayManagerTest, UIScaleWithDisplayMode) {
@@ -2796,16 +2783,6 @@ TEST_F(DisplayManagerFontTest, TextSubpixelPositioningWithDsf100Internal) {
EXPECT_NE(gfx::FontRenderParams::HINTING_NONE, GetFontHintingParams());
}
-TEST_F(DisplayManagerFontTest, TextSubpixelPositioningWithDsf125Internal) {
- display::test::ScopedDisable125DSFForUIScaling disable;
- FontTestHelper helper(1.25f, FontTestHelper::INTERNAL);
- ASSERT_DOUBLE_EQ(
- 1.25f,
- display::Screen::GetScreen()->GetPrimaryDisplay().device_scale_factor());
- EXPECT_TRUE(IsTextSubpixelPositioningEnabled());
- EXPECT_EQ(gfx::FontRenderParams::HINTING_NONE, GetFontHintingParams());
-}
-
TEST_F(DisplayManagerFontTest, TextSubpixelPositioningWithDsf200Internal) {
FontTestHelper helper(2.0f, FontTestHelper::INTERNAL);
ASSERT_DOUBLE_EQ(
« no previous file with comments | « no previous file | ui/display/manager/chromeos/display_change_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698