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

Unified Diff: ash/display/window_tree_host_manager.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: . 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
Index: ash/display/window_tree_host_manager.cc
diff --git a/ash/display/window_tree_host_manager.cc b/ash/display/window_tree_host_manager.cc
index 70dd1b9b521a41262bb8878dac54ecfb5218a452..5a561b68cb5642f3551983b1a643bd0e8d5e1822 100644
--- a/ash/display/window_tree_host_manager.cc
+++ b/ash/display/window_tree_host_manager.cc
@@ -282,7 +282,7 @@ WindowTreeHostManager::WindowTreeHostManager()
WindowTreeHostManager::~WindowTreeHostManager() {}
void WindowTreeHostManager::Start() {
- Shell::GetScreen()->AddObserver(this);
+ gfx::Screen::GetScreen()->AddObserver(this);
Shell::GetInstance()->display_manager()->set_delegate(this);
}
@@ -296,9 +296,9 @@ void WindowTreeHostManager::Shutdown() {
cursor_window_controller_.reset();
mirror_window_controller_.reset();
- Shell::GetScreen()->RemoveObserver(this);
+ gfx::Screen::GetScreen()->RemoveObserver(this);
- int64_t primary_id = Shell::GetScreen()->GetPrimaryDisplay().id();
+ int64_t primary_id = gfx::Screen::GetScreen()->GetPrimaryDisplay().id();
// Delete non primary root window controllers first, then
// delete the primary root window controller.
@@ -452,7 +452,7 @@ void WindowTreeHostManager::SwapPrimaryDisplay() {
limiter_->SetThrottleTimeout(kSwapDisplayThrottleTimeoutMs);
}
- if (Shell::GetScreen()->GetNumDisplays() > 1) {
+ if (gfx::Screen::GetScreen()->GetNumDisplays() > 1) {
#if defined(OS_CHROMEOS)
DisplayAnimator* animation = Shell::GetInstance()->display_animator();
if (animation) {
@@ -504,7 +504,8 @@ void WindowTreeHostManager::SetPrimaryDisplay(
if (!non_primary_host)
return;
- gfx::Display old_primary_display = Shell::GetScreen()->GetPrimaryDisplay();
+ gfx::Display old_primary_display =
+ gfx::Screen::GetScreen()->GetPrimaryDisplay();
// Swap root windows between current and new primary display.
AshWindowTreeHost* primary_host = window_tree_hosts_[primary_display_id];
@@ -544,7 +545,7 @@ void WindowTreeHostManager::UpdateMouseLocationAfterDisplayChange() {
// use the same native location. Otherwise find the display closest
// to the current cursor location in screen coordinates.
- gfx::Point point_in_screen = Shell::GetScreen()->GetCursorScreenPoint();
+ gfx::Point point_in_screen = gfx::Screen::GetScreen()->GetCursorScreenPoint();
gfx::Point target_location_in_native;
int64_t closest_distance_squared = -1;
DisplayManager* display_manager = GetDisplayManager();
@@ -784,7 +785,7 @@ void WindowTreeHostManager::OnDisplayMetricsChanged(const gfx::Display& display,
}
void WindowTreeHostManager::OnHostResized(const aura::WindowTreeHost* host) {
- gfx::Display display = Shell::GetScreen()->GetDisplayNearestWindow(
+ gfx::Display display = gfx::Screen::GetScreen()->GetDisplayNearestWindow(
const_cast<aura::Window*>(host->window()));
DisplayManager* display_manager = GetDisplayManager();
@@ -818,7 +819,7 @@ void WindowTreeHostManager::CloseMirroringDisplayIfNotNecessary() {
void WindowTreeHostManager::PreDisplayConfigurationChange(bool clear_focus) {
FOR_EACH_OBSERVER(Observer, observers_, OnDisplayConfigurationChanging());
focus_activation_store_->Store(clear_focus);
- gfx::Screen* screen = Shell::GetScreen();
+ gfx::Screen* screen = gfx::Screen::GetScreen();
gfx::Point point_in_screen = screen->GetCursorScreenPoint();
cursor_location_in_screen_coords_for_restore_ = point_in_screen;
@@ -846,7 +847,7 @@ void WindowTreeHostManager::PostDisplayConfigurationChange() {
layout_store->UpdateMultiDisplayState(
pair, display_manager->IsInMirrorMode(), layout.default_unified);
- if (Shell::GetScreen()->GetNumDisplays() > 1) {
+ if (gfx::Screen::GetScreen()->GetNumDisplays() > 1) {
int64_t primary_id = layout.primary_id;
SetPrimaryDisplayId(primary_id == gfx::Display::kInvalidDisplayID
? pair.first
@@ -856,7 +857,7 @@ void WindowTreeHostManager::PostDisplayConfigurationChange() {
// doesn't exist, or b) the primary_id has already been
// set to the same and didn't update it.
layout_store->UpdatePrimaryDisplayId(
- pair, Shell::GetScreen()->GetPrimaryDisplay().id());
+ pair, gfx::Screen::GetScreen()->GetPrimaryDisplay().id());
}
}
FOR_EACH_OBSERVER(Observer, observers_, OnDisplayConfigurationChanged());

Powered by Google App Engine
This is Rietveld 408576698