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

Unified Diff: ash/wm/shelf_layout_manager_unittest.cc

Issue 9960042: Refactor screen/monitor so that gfx::Screen returns monitor object. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix command line Created 8 years, 8 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/wm/shelf_layout_manager_unittest.cc
diff --git a/ash/wm/shelf_layout_manager_unittest.cc b/ash/wm/shelf_layout_manager_unittest.cc
index 7ab0637f576492de3254610cd3f06ceb94d5021c..1c2a52e59af953f8b4694d2f205713c4344fbee7 100644
--- a/ash/wm/shelf_layout_manager_unittest.cc
+++ b/ash/wm/shelf_layout_manager_unittest.cc
@@ -13,7 +13,7 @@
#include "ash/test/ash_test_base.h"
#include "ui/aura/client/aura_constants.h"
#include "ui/aura/env.h"
-#include "ui/aura/monitor.h"
+#include "ui/aura/monitor_aura.h"
#include "ui/aura/monitor_manager.h"
#include "ui/aura/root_window.h"
#include "ui/aura/test/event_generator.h"
@@ -82,7 +82,7 @@ TEST_F(ShelfLayoutManagerTest, MAYBE_SetVisible) {
EXPECT_EQ(ShelfLayoutManager::VISIBLE, shelf->visibility_state());
const aura::MonitorManager* manager =
aura::Env::GetInstance()->monitor_manager();
- const aura::Monitor* monitor =
+ const aura::MonitorAura* monitor =
manager->GetMonitorNearestWindow(Shell::GetRootWindow());
ASSERT_TRUE(monitor);
// Bottom inset should be the max of widget heights.
@@ -99,9 +99,9 @@ TEST_F(ShelfLayoutManagerTest, MAYBE_SetVisible) {
// Make sure the bounds of the two widgets changed.
EXPECT_GE(shelf->launcher_widget()->GetNativeView()->bounds().y(),
- gfx::Screen::GetPrimaryMonitorBounds().bottom());
+ gfx::Screen::GetPrimaryMonitor()->GetBounds().bottom());
EXPECT_GE(shelf->status()->GetNativeView()->bounds().y(),
- gfx::Screen::GetPrimaryMonitorBounds().bottom());
+ gfx::Screen::GetPrimaryMonitor()->GetBounds().bottom());
// And show it again.
SetState(shelf, ShelfLayoutManager::VISIBLE);
@@ -115,7 +115,7 @@ TEST_F(ShelfLayoutManagerTest, MAYBE_SetVisible) {
// Make sure the bounds of the two widgets changed.
gfx::Rect launcher_bounds(
shelf->launcher_widget()->GetNativeView()->bounds());
- int bottom = gfx::Screen::GetPrimaryMonitorBounds().bottom() -
+ int bottom = gfx::Screen::GetPrimaryMonitor()->GetBounds().bottom() -
shelf->shelf_height();
EXPECT_EQ(launcher_bounds.y(),
bottom + (shelf->shelf_height() - launcher_bounds.height()) / 2);
@@ -133,7 +133,7 @@ TEST_F(ShelfLayoutManagerTest, LayoutShelfWhileAnimating) {
const aura::MonitorManager* manager =
aura::Env::GetInstance()->monitor_manager();
- const aura::Monitor* monitor =
+ const aura::MonitorAura* monitor =
manager->GetMonitorNearestWindow(Shell::GetRootWindow());
// Hide the shelf.
@@ -143,9 +143,9 @@ TEST_F(ShelfLayoutManagerTest, LayoutShelfWhileAnimating) {
EXPECT_EQ(0, monitor->work_area_insets().bottom());
// Make sure the bounds of the two widgets changed.
EXPECT_GE(shelf->launcher_widget()->GetNativeView()->bounds().y(),
- gfx::Screen::GetPrimaryMonitorBounds().bottom());
+ gfx::Screen::GetPrimaryMonitor()->GetBounds().bottom());
EXPECT_GE(shelf->status()->GetNativeView()->bounds().y(),
- gfx::Screen::GetPrimaryMonitorBounds().bottom());
+ gfx::Screen::GetPrimaryMonitor()->GetBounds().bottom());
}
// Makes sure the launcher is initially sized correctly.

Powered by Google App Engine
This is Rietveld 408576698