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

Unified Diff: ash/display/display_manager_unittest.cc

Issue 72503002: Remove some pass-thrus on RootWindow API in favor of exposing the RootWindowHost again. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
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
« no previous file with comments | « ash/display/display_controller_unittest.cc ('k') | ash/display/mirror_window_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/display/display_manager_unittest.cc
diff --git a/ash/display/display_manager_unittest.cc b/ash/display/display_manager_unittest.cc
index 5f33d5a6af0c02ac61a79eb134821987ded4534c..f6bd59ee6f0dad708b8a8b2f5217ee1b31df5e5a 100644
--- a/ash/display/display_manager_unittest.cc
+++ b/ash/display/display_manager_unittest.cc
@@ -591,7 +591,7 @@ TEST_F(DisplayManagerTest, MAYBE_TestNativeDisplaysChangedNoInternal) {
EXPECT_EQ("1,1 100x100",
GetDisplayInfoForId(10).bounds_in_native().ToString());
EXPECT_EQ("100x100", ash::Shell::GetPrimaryRootWindow()->GetDispatcher()->
- GetHostSize().ToString());
+ host()->GetBounds().size().ToString());
}
#if defined(OS_WIN)
@@ -979,7 +979,8 @@ TEST_F(DisplayManagerTest, SoftwareMirroring) {
EXPECT_EQ(1U, display_manager->GetNumDisplays());
EXPECT_EQ("0,0 300x400",
Shell::GetScreen()->GetPrimaryDisplay().bounds().ToString());
- EXPECT_EQ("400x500", test_api.GetRootWindow()->GetHostSize().ToString());
+ EXPECT_EQ("400x500",
+ test_api.GetRootWindow()->host()->GetBounds().size().ToString());
EXPECT_EQ("300x400",
test_api.GetRootWindow()->window()->bounds().size().ToString());
EXPECT_TRUE(display_manager->IsMirrored());
@@ -1092,32 +1093,32 @@ TEST_F(DisplayManagerTest, MAYBE_UpdateDisplayWithHostOrigin) {
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", dispatcher0->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("100x200", dispatcher0->host()->GetBounds().size().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", dispatcher1->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("300x400", dispatcher1->host()->GetBounds().size().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", dispatcher0->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("100x200", dispatcher0->host()->GetBounds().size().ToString());
+ EXPECT_EQ("200,300", dispatcher1->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("300x400", dispatcher1->host()->GetBounds().size().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", dispatcher0->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("200x300", dispatcher0->host()->GetBounds().size().ToString());
+ EXPECT_EQ("0,0", dispatcher1->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("300x400", dispatcher1->host()->GetBounds().size().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", dispatcher0->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("100x200", dispatcher0->host()->GetBounds().size().ToString());
+ EXPECT_EQ("300,500", dispatcher1->host()->GetBounds().origin().ToString());
+ EXPECT_EQ("200x300", dispatcher1->host()->GetBounds().size().ToString());
}
} // namespace internal
« no previous file with comments | « ash/display/display_controller_unittest.cc ('k') | ash/display/mirror_window_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698