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

Unified Diff: ash/display/multi_display_manager_unittest.cc

Issue 10909043: Cancel drag if display configuration changes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: win fix, adjust test screen size Created 8 years, 3 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: ash/display/multi_display_manager_unittest.cc
diff --git a/ash/display/multi_display_manager_unittest.cc b/ash/display/multi_display_manager_unittest.cc
index 401c5266fd56c9610a20535b53f3e4fdd144876a..83c5cab97fb98ce867d2395b6ee5bf8f9d2dbdad 100644
--- a/ash/display/multi_display_manager_unittest.cc
+++ b/ash/display/multi_display_manager_unittest.cc
@@ -195,29 +195,20 @@ TEST_F(MultiDisplayManagerTest, MAYBE_NativeDisplayTest) {
TEST_F(MultiDisplayManagerTest, MAYBE_EmulatorTest) {
EXPECT_EQ(1U, display_manager()->GetNumDisplays());
- internal::MultiDisplayManager::AddRemoveDisplay();
+ internal::MultiDisplayManager::CycleDisplay();
// Update primary and add seconary.
EXPECT_EQ(2U, display_manager()->GetNumDisplays());
- EXPECT_EQ("1 1 0", GetCountSummary());
+ EXPECT_EQ("0 1 0", GetCountSummary());
reset();
internal::MultiDisplayManager::CycleDisplay();
- EXPECT_EQ(2U, display_manager()->GetNumDisplays());
- // Observer gets called twice in this mode because
- // it gets notified both from |OnNativeDisplayChagned|
- // and from |RootWindowObserver|, which is the consequence of
- // |SetHostSize()|.
- EXPECT_EQ("4 0 0", GetCountSummary());
- reset();
-
- internal::MultiDisplayManager::AddRemoveDisplay();
EXPECT_EQ(1U, display_manager()->GetNumDisplays());
EXPECT_EQ("0 0 1", GetCountSummary());
reset();
internal::MultiDisplayManager::CycleDisplay();
- EXPECT_EQ(1U, display_manager()->GetNumDisplays());
- EXPECT_EQ("0 0 0", GetCountSummary());
+ EXPECT_EQ(2U, display_manager()->GetNumDisplays());
+ EXPECT_EQ("0 1 0", GetCountSummary());
reset();
}

Powered by Google App Engine
This is Rietveld 408576698