Index: chrome/browser/chromeos/display/display_preferences_unittest.cc |
diff --git a/chrome/browser/chromeos/display/display_preferences_unittest.cc b/chrome/browser/chromeos/display/display_preferences_unittest.cc |
index 05c19b7465016a7a3f52ba133b42ac7c1e45b9c9..7a0f9988ac0bdebe140cde9d6c82d66a3b0b6b74 100644 |
--- a/chrome/browser/chromeos/display/display_preferences_unittest.cc |
+++ b/chrome/browser/chromeos/display/display_preferences_unittest.cc |
@@ -199,7 +199,7 @@ TEST_F(DisplayPreferencesTest, BasicStores) { |
ash::DisplayManager* display_manager = |
ash::Shell::GetInstance()->display_manager(); |
- UpdateDisplay("200x200*2, 400x300#400x400|300x200"); |
+ UpdateDisplay("200x200*2, 400x300#400x400|300x200*1.25"); |
int64 id1 = gfx::Screen::GetNativeScreen()->GetPrimaryDisplay().id(); |
gfx::Display::SetInternalDisplayId(id1); |
int64 id2 = ash::ScreenUtil::GetSecondaryDisplay().id(); |
@@ -292,13 +292,15 @@ TEST_F(DisplayPreferencesTest, BasicStores) { |
EXPECT_FALSE(property->GetString("color_profile_name", &color_profile)); |
// Resolution is saved only when the resolution is set |
- // by DisplayManager::SetDisplayResolution |
+ // by DisplayManager::SetDisplayMode |
width = 0; |
height = 0; |
EXPECT_FALSE(property->GetInteger("width", &width)); |
EXPECT_FALSE(property->GetInteger("height", &height)); |
- display_manager->SetDisplayResolution(id2, gfx::Size(300, 200)); |
+ ash::DisplayMode mode(gfx::Size(300, 200), 60.0f, false, true); |
+ mode.device_scale_factor = 1.25f; |
+ display_manager->SetDisplayMode(id2, mode); |
display_controller->SetPrimaryDisplayId(id2); |
@@ -311,11 +313,15 @@ TEST_F(DisplayPreferencesTest, BasicStores) { |
// External display's resolution must be stored this time because |
// it's not best. |
+ int device_scale_factor = 0; |
EXPECT_TRUE(properties->GetDictionary(base::Int64ToString(id2), &property)); |
EXPECT_TRUE(property->GetInteger("width", &width)); |
EXPECT_TRUE(property->GetInteger("height", &height)); |
+ EXPECT_TRUE(property->GetInteger( |
+ "device-scale-factor", &device_scale_factor)); |
EXPECT_EQ(300, width); |
EXPECT_EQ(200, height); |
+ EXPECT_EQ(1250, device_scale_factor); |
// The layout remains the same. |
EXPECT_TRUE(displays->GetDictionary(key, &layout_value)); |
@@ -364,7 +370,7 @@ TEST_F(DisplayPreferencesTest, BasicStores) { |
// Set new display's selected resolution. |
display_manager->RegisterDisplayProperty( |
- id2 + 1, gfx::Display::ROTATE_0, 1.0f, NULL, gfx::Size(500, 400), |
+ id2 + 1, gfx::Display::ROTATE_0, 1.0f, NULL, gfx::Size(500, 400), 1.0f, |
ui::COLOR_PROFILE_STANDARD); |
UpdateDisplay("200x200*2, 600x500#600x500|500x400"); |
@@ -390,7 +396,7 @@ TEST_F(DisplayPreferencesTest, BasicStores) { |
// Set yet another new display's selected resolution. |
display_manager->RegisterDisplayProperty( |
- id2 + 1, gfx::Display::ROTATE_0, 1.0f, NULL, gfx::Size(500, 400), |
+ id2 + 1, gfx::Display::ROTATE_0, 1.0f, NULL, gfx::Size(500, 400), 1.0f, |
ui::COLOR_PROFILE_STANDARD); |
// Disconnect 2nd display first to generate new id for external display. |
UpdateDisplay("200x200*2"); |
@@ -452,9 +458,9 @@ TEST_F(DisplayPreferencesTest, PreventStore) { |
ResolutionNotificationController::kNotificationId)); |
// Once the notification is removed, the specified resolution will be stored |
- // by SetDisplayResolution. |
- ash::Shell::GetInstance()->display_manager()->SetDisplayResolution( |
- id, gfx::Size(300, 200)); |
+ // by SetDisplayMode. |
+ ash::Shell::GetInstance()->display_manager()->SetDisplayMode( |
+ id, ash::DisplayMode(gfx::Size(300, 200), 60.0f, false, true)); |
UpdateDisplay("300x200#500x400|400x300|300x200"); |
property = NULL; |