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

Unified Diff: ash/wm/lock_layout_manager_unittest.cc

Issue 1608053002: Remove now-unnecessary wrappers around gfx::Screen::GetScreen() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@metro-mode-5
Patch Set: cros Created 4 years, 11 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
« no previous file with comments | « ash/wm/immersive_fullscreen_controller.cc ('k') | ash/wm/maximize_mode/maximize_mode_window_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/wm/lock_layout_manager_unittest.cc
diff --git a/ash/wm/lock_layout_manager_unittest.cc b/ash/wm/lock_layout_manager_unittest.cc
index 5439a7269bbefec908b4ff6b23510a2293655474..9d56ab71e427519f80df4667c4e24c95160bab1d 100644
--- a/ash/wm/lock_layout_manager_unittest.cc
+++ b/ash/wm/lock_layout_manager_unittest.cc
@@ -105,7 +105,8 @@ class LockLayoutManagerTest : public AshTestBase {
};
TEST_F(LockLayoutManagerTest, NorwmalWindowBoundsArePreserved) {
- gfx::Rect screen_bounds = Shell::GetScreen()->GetPrimaryDisplay().bounds();
+ gfx::Rect screen_bounds =
+ gfx::Screen::GetScreen()->GetPrimaryDisplay().bounds();
views::Widget::InitParams widget_params(
views::Widget::InitParams::TYPE_WINDOW);
@@ -131,7 +132,8 @@ TEST_F(LockLayoutManagerTest, MaximizedFullscreenWindowBoundsAreEqualToScreen) {
if (!SupportsHostWindowResize())
return;
- gfx::Rect screen_bounds = Shell::GetScreen()->GetPrimaryDisplay().bounds();
+ gfx::Rect screen_bounds =
+ gfx::Screen::GetScreen()->GetPrimaryDisplay().bounds();
views::Widget::InitParams widget_params(
views::Widget::InitParams::TYPE_WINDOW_FRAMELESS);
@@ -183,7 +185,7 @@ TEST_F(LockLayoutManagerTest, KeyboardBounds) {
if (!SupportsHostWindowResize())
return;
- gfx::Display primary_display = Shell::GetScreen()->GetPrimaryDisplay();
+ gfx::Display primary_display = gfx::Screen::GetScreen()->GetPrimaryDisplay();
gfx::Rect screen_bounds = primary_display.bounds();
views::Widget::InitParams widget_params(
@@ -219,7 +221,7 @@ TEST_F(LockLayoutManagerTest, KeyboardBounds) {
display_manager->SetDisplayRotation(primary_display.id(),
gfx::Display::ROTATE_90,
gfx::Display::ROTATION_SOURCE_ACTIVE);
- primary_display = Shell::GetScreen()->GetPrimaryDisplay();
+ primary_display = gfx::Screen::GetScreen()->GetPrimaryDisplay();
screen_bounds = primary_display.bounds();
EXPECT_EQ(screen_bounds.ToString(), window->GetBoundsInScreen().ToString());
display_manager->SetDisplayRotation(primary_display.id(),
@@ -233,7 +235,7 @@ TEST_F(LockLayoutManagerTest, KeyboardBounds) {
ShowKeyboard(true);
keyboard::KeyboardController* keyboard =
keyboard::KeyboardController::GetInstance();
- primary_display = Shell::GetScreen()->GetPrimaryDisplay();
+ primary_display = gfx::Screen::GetScreen()->GetPrimaryDisplay();
screen_bounds = primary_display.bounds();
gfx::Rect target_bounds(screen_bounds);
target_bounds.set_height(
@@ -251,7 +253,8 @@ TEST_F(LockLayoutManagerTest, MultipleMonitors) {
return;
UpdateDisplay("300x400,400x500");
- gfx::Rect screen_bounds = Shell::GetScreen()->GetPrimaryDisplay().bounds();
+ gfx::Rect screen_bounds =
+ gfx::Screen::GetScreen()->GetPrimaryDisplay().bounds();
aura::Window::Windows root_windows = Shell::GetAllRootWindows();
views::Widget::InitParams widget_params(
« no previous file with comments | « ash/wm/immersive_fullscreen_controller.cc ('k') | ash/wm/maximize_mode/maximize_mode_window_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698