diff options
author | oshima <oshima@chromium.org> | 2015-07-31 18:33:14 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-08-01 01:34:05 +0000 |
commit | 628a617979598ad6ade68cc7f48815477a2f903a (patch) | |
tree | c2ff77b8cc668c6afa400c7f714f6b4f7764daa6 /chrome/browser/chromeos/display/display_preferences_unittest.cc | |
parent | 47d6636cd48907043fd615a51dc00af379db19fe (diff) | |
download | chromium_src-628a617979598ad6ade68cc7f48815477a2f903a.zip chromium_src-628a617979598ad6ade68cc7f48815477a2f903a.tar.gz chromium_src-628a617979598ad6ade68cc7f48815477a2f903a.tar.bz2 |
Allow dynamic enabling/disabling of unified desktop
The default multi display is stored in the layout store first, and which is used by UpdateDisplays later.
Hence no need to update this layout information in
WindowTreeHostManager. current_default_multi_display_mode_ is still necessary because it's also used when switching to/from mirror mode.
This also fixed a bug in the logic to short ids.
BUG=515607
TEST=covered by unittests. manual
Review URL: https://codereview.chromium.org/1261693004
Cr-Commit-Position: refs/heads/master@{#341452}
Diffstat (limited to 'chrome/browser/chromeos/display/display_preferences_unittest.cc')
-rw-r--r-- | chrome/browser/chromeos/display/display_preferences_unittest.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/chromeos/display/display_preferences_unittest.cc b/chrome/browser/chromeos/display/display_preferences_unittest.cc index eb065ad..07c0514 100644 --- a/chrome/browser/chromeos/display/display_preferences_unittest.cc +++ b/chrome/browser/chromeos/display/display_preferences_unittest.cc @@ -876,11 +876,11 @@ TEST_F(DisplayPreferencesTest, RotationLockTriggersStore) { } TEST_F(DisplayPreferencesTest, SaveUnifiedMode) { - ash::test::DisplayManagerTestApi::EnableUnifiedDesktopForTest(); LoggedInAsUser(); ash::DisplayManager* display_manager = ash::Shell::GetInstance()->display_manager(); + display_manager->SetUnifiedDesktopEnabled(true); UpdateDisplay("200x200,100x100"); ash::DisplayIdPair pair = display_manager->GetCurrentDisplayIdPair(); @@ -928,7 +928,8 @@ TEST_F(DisplayPreferencesTest, SaveUnifiedMode) { EXPECT_FALSE(stored_layout.mirrored); // Exit unified mode. - display_manager->SetDefaultMultiDisplayMode(ash::DisplayManager::EXTENDED); + display_manager->SetDefaultMultiDisplayModeForCurrentDisplays( + ash::DisplayManager::EXTENDED); display_manager->ReconfigureDisplays(); ASSERT_TRUE( secondary_displays->GetDictionary(ToPairString(pair), &new_value)); @@ -953,7 +954,7 @@ TEST_F(DisplayPreferencesTest, RestoreUnifiedMode) { EXPECT_FALSE(display_manager->IsInUnifiedMode()); // Restored to unified. - ash::test::DisplayManagerTestApi::EnableUnifiedDesktopForTest(); + display_manager->SetUnifiedDesktopEnabled(true); StoreDisplayBoolPropertyForPair(pair, "default_unified", true); LoadDisplayPreferences(false); UpdateDisplay("100x100,200x200"); |