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

Unified Diff: chrome/browser/chromeos/login/ui/login_display_host_impl.cc

Issue 2885153004: React to primary display change instead of removal (Closed)
Patch Set: Minor fixes and documentation changes Created 3 years, 7 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: chrome/browser/chromeos/login/ui/login_display_host_impl.cc
diff --git a/chrome/browser/chromeos/login/ui/login_display_host_impl.cc b/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
index a498e861c9d909c276d7263028ab47c2232d2eb3..80a6daa3da0bbc0bae92f81ce8820f527b5b5d78 100644
--- a/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
+++ b/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
@@ -980,16 +980,10 @@ void LoginDisplayHostImpl::OnDisplayAdded(const display::Display& new_display) {
GetOobeUI()->OnDisplayConfigurationChanged();
}
-void LoginDisplayHostImpl::OnDisplayRemoved(
- const display::Display& old_display) {
- if (GetOobeUI())
- GetOobeUI()->OnDisplayConfigurationChanged();
-}
-
void LoginDisplayHostImpl::OnDisplayMetricsChanged(
const display::Display& display,
uint32_t changed_metrics) {
- display::Display primary_display =
+ const display::Display primary_display =
display::Screen::GetScreen()->GetPrimaryDisplay();
if (display.id() != primary_display.id() ||
!(changed_metrics & DISPLAY_METRIC_BOUNDS)) {
@@ -1000,6 +994,9 @@ void LoginDisplayHostImpl::OnDisplayMetricsChanged(
const gfx::Size& size = primary_display.size();
GetOobeUI()->GetCoreOobeView()->SetClientAreaSize(size.width(),
size.height());
+
+ if (changed_metrics & DISPLAY_METRIC_PRIMARY)
+ GetOobeUI()->OnDisplayConfigurationChanged();
}
}

Powered by Google App Engine
This is Rietveld 408576698