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

Unified Diff: chrome/browser/chromeos/display/display_preferences_unittest.cc

Issue 1594683002: Introduce DisplayConfigurationController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue_576375_display1b1
Patch Set: DisplayAnimator -> DisplayAnimatorChromeOS Created 4 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
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 1c93b180a35a0be84dfeca19d28a0a20fb57198e..24150bb7b3fb6e3c5ef87e9ca4147399e7e75668 100644
--- a/chrome/browser/chromeos/display/display_preferences_unittest.cc
+++ b/chrome/browser/chromeos/display/display_preferences_unittest.cc
@@ -264,7 +264,7 @@ TEST_F(DisplayPreferencesTest, BasicStores) {
LoggedInAsUser();
ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
- SetCurrentDisplayLayout(layout);
+ display_manager->SetLayoutForCurrentDisplays(layout);
StoreDisplayLayoutPrefForTest(
id1, dummy_id, ash::DisplayLayout(ash::DisplayLayout::LEFT, 20));
// Can't switch to a display that does not exist.
@@ -385,7 +385,7 @@ TEST_F(DisplayPreferencesTest, BasicStores) {
EXPECT_TRUE(layout_value->GetString(kPrimaryIdKey, &primary_id_str));
EXPECT_EQ(base::Int64ToString(id2), primary_id_str);
- SetCurrentDisplayLayout(
+ display_manager->SetLayoutForCurrentDisplays(
ash::DisplayLayout(ash::DisplayLayout::BOTTOM, 20));
UpdateDisplay("1+0-200x200*2,1+0-200x200");
@@ -530,7 +530,8 @@ TEST_F(DisplayPreferencesTest, StoreForSwappedDisplay) {
LoggedInAsUser();
ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
- SetCurrentDisplayLayout(layout);
+ ash::Shell::GetInstance()->display_manager()->SetLayoutForCurrentDisplays(
+ layout);
layout = layout.Invert();
const base::DictionaryValue* displays =
@@ -592,10 +593,10 @@ TEST_F(DisplayPreferencesTest, DontStoreInGuestMode) {
int64_t id1 = ash::Shell::GetScreen()->GetPrimaryDisplay().id();
ash::test::ScopedSetInternalDisplayId set_internal(id1);
int64_t id2 = ash::ScreenUtil::GetSecondaryDisplay().id();
- ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
- SetCurrentDisplayLayout(layout);
ash::DisplayManager* display_manager =
ash::Shell::GetInstance()->display_manager();
+ ash::DisplayLayout layout(ash::DisplayLayout::TOP, 10);
+ display_manager->SetLayoutForCurrentDisplays(layout);
ash::SetDisplayUIScale(id1, 1.25f);
window_tree_host_manager->SetPrimaryDisplayId(id2);
int64_t new_primary = ash::Shell::GetScreen()->GetPrimaryDisplay().id();
@@ -933,7 +934,6 @@ TEST_F(DisplayPreferencesTest, SaveUnifiedMode) {
// Exit unified mode.
display_manager->SetDefaultMultiDisplayModeForCurrentDisplays(
ash::DisplayManager::EXTENDED);
- display_manager->ReconfigureDisplays();
ASSERT_TRUE(
secondary_displays->GetDictionary(ToPairString(pair), &new_value));
EXPECT_TRUE(ash::DisplayLayout::ConvertFromValue(*new_value, &stored_layout));

Powered by Google App Engine
This is Rietveld 408576698