Chromium Code Reviews| 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..bfe9f8d107d3ca06d2a9e6658d63857ad043bf11 100644 |
| --- a/ash/display/display_manager_unittest.cc |
| +++ b/ash/display/display_manager_unittest.cc |
| @@ -1673,7 +1673,7 @@ TEST_F(DisplayManagerTest, Rotate) { |
| } |
| TEST_F(DisplayManagerTest, UIScale) { |
| - display::test::ScopedDisable125DSFForUIScaling disable; |
| + // display::test::ScopedDisable125DSFForUIScaling disable; |
|
afakhry
2017/06/20 19:01:09
Nit: Please remove this line.
oshima
2017/06/20 20:06:41
Done.
|
| UpdateDisplay("1280x800"); |
| int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id(); |
| @@ -1785,17 +1785,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 +2785,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( |