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

Unified Diff: trunk/src/ash/display/display_manager_unittest.cc

Issue 59153004: Revert 233787 "Eliminate Shell::RootWindowList in favor of aura:..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month 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: trunk/src/ash/display/display_manager_unittest.cc
===================================================================
--- trunk/src/ash/display/display_manager_unittest.cc (revision 233812)
+++ trunk/src/ash/display/display_manager_unittest.cc (working copy)
@@ -605,7 +605,7 @@
TEST_F(DisplayManagerTest, MAYBE_EnsurePointerInDisplays) {
UpdateDisplay("200x200,300x300");
- aura::Window::Windows root_windows = Shell::GetAllRootWindows();
+ Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
aura::Env* env = aura::Env::GetInstance();
@@ -652,7 +652,7 @@
layout_store->SetDefaultDisplayLayout(layout);
UpdateDisplay("200x200,300x300");
- aura::Window::Windows root_windows = Shell::GetAllRootWindows();
+ Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
EXPECT_EQ("-300,0 300x300",
ScreenAsh::GetSecondaryDisplay().bounds().ToString());
@@ -881,7 +881,7 @@
TEST_F(DisplayManagerTest, MAYBE_UpdateMouseCursorAfterRotateZoom) {
// Make sure just rotating will not change native location.
UpdateDisplay("300x200,200x150");
- aura::Window::Windows root_windows = Shell::GetAllRootWindows();
+ Shell::RootWindowList root_windows = Shell::GetAllRootWindows();
aura::Env* env = aura::Env::GetInstance();
aura::test::EventGenerator generator1(root_windows[0]);
@@ -1080,38 +1080,35 @@
TEST_F(DisplayManagerTest, MAYBE_UpdateDisplayWithHostOrigin) {
UpdateDisplay("100x200,300x400");
ASSERT_EQ(2, Shell::GetScreen()->GetNumDisplays());
- aura::Window::Windows root_windows =
+ Shell::RootWindowList root_windows =
Shell::GetInstance()->GetAllRootWindows();
ASSERT_EQ(2U, root_windows.size());
- aura::WindowEventDispatcher* dispatcher0 = root_windows[0]->GetDispatcher();
- aura::WindowEventDispatcher* dispatcher1 = root_windows[1]->GetDispatcher();
-
- EXPECT_EQ("1,1", dispatcher0->GetHostOrigin().ToString());
- EXPECT_EQ("100x200", dispatcher0->GetHostSize().ToString());
+ EXPECT_EQ("1,1", root_windows[0]->GetHostOrigin().ToString());
+ EXPECT_EQ("100x200", root_windows[0]->GetHostSize().ToString());
// UpdateDisplay set the origin if it's not set.
- EXPECT_NE("1,1", dispatcher1->GetHostOrigin().ToString());
- EXPECT_EQ("300x400", dispatcher1->GetHostSize().ToString());
+ EXPECT_NE("1,1", root_windows[1]->GetHostOrigin().ToString());
+ EXPECT_EQ("300x400", root_windows[1]->GetHostSize().ToString());
UpdateDisplay("100x200,200+300-300x400");
ASSERT_EQ(2, Shell::GetScreen()->GetNumDisplays());
- EXPECT_EQ("0,0", dispatcher0->GetHostOrigin().ToString());
- EXPECT_EQ("100x200", dispatcher0->GetHostSize().ToString());
- EXPECT_EQ("200,300", dispatcher1->GetHostOrigin().ToString());
- EXPECT_EQ("300x400", dispatcher1->GetHostSize().ToString());
+ EXPECT_EQ("0,0", root_windows[0]->GetHostOrigin().ToString());
+ EXPECT_EQ("100x200", root_windows[0]->GetHostSize().ToString());
+ EXPECT_EQ("200,300", root_windows[1]->GetHostOrigin().ToString());
+ EXPECT_EQ("300x400", root_windows[1]->GetHostSize().ToString());
UpdateDisplay("400+500-200x300,300x400");
ASSERT_EQ(2, Shell::GetScreen()->GetNumDisplays());
- EXPECT_EQ("400,500", dispatcher0->GetHostOrigin().ToString());
- EXPECT_EQ("200x300", dispatcher0->GetHostSize().ToString());
- EXPECT_EQ("0,0", dispatcher1->GetHostOrigin().ToString());
- EXPECT_EQ("300x400", dispatcher1->GetHostSize().ToString());
+ EXPECT_EQ("400,500", root_windows[0]->GetHostOrigin().ToString());
+ EXPECT_EQ("200x300", root_windows[0]->GetHostSize().ToString());
+ EXPECT_EQ("0,0", root_windows[1]->GetHostOrigin().ToString());
+ EXPECT_EQ("300x400", root_windows[1]->GetHostSize().ToString());
UpdateDisplay("100+200-100x200,300+500-200x300");
ASSERT_EQ(2, Shell::GetScreen()->GetNumDisplays());
- EXPECT_EQ("100,200", dispatcher0->GetHostOrigin().ToString());
- EXPECT_EQ("100x200", dispatcher0->GetHostSize().ToString());
- EXPECT_EQ("300,500", dispatcher1->GetHostOrigin().ToString());
- EXPECT_EQ("200x300", dispatcher1->GetHostSize().ToString());
+ EXPECT_EQ("100,200", root_windows[0]->GetHostOrigin().ToString());
+ EXPECT_EQ("100x200", root_windows[0]->GetHostSize().ToString());
+ EXPECT_EQ("300,500", root_windows[1]->GetHostOrigin().ToString());
+ EXPECT_EQ("200x300", root_windows[1]->GetHostSize().ToString());
}
} // namespace internal
« no previous file with comments | « trunk/src/ash/display/display_controller_unittest.cc ('k') | trunk/src/ash/display/mouse_cursor_event_filter_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698