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

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: Workaround DisplayConfigurationController display switch limiter 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..fd221dddf564cbe819704d9cf3197451379db026 100644
--- a/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
+++ b/chrome/browser/chromeos/login/ui/login_display_host_impl.cc
@@ -981,10 +981,7 @@ void LoginDisplayHostImpl::OnDisplayAdded(const display::Display& new_display) {
}
void LoginDisplayHostImpl::OnDisplayRemoved(
- const display::Display& old_display) {
- if (GetOobeUI())
- GetOobeUI()->OnDisplayConfigurationChanged();
-}
+ const display::Display& old_display) {}
oshima 2017/05/17 16:56:18 you can remove this
void LoginDisplayHostImpl::OnDisplayMetricsChanged(
const display::Display& display,
@@ -1000,6 +997,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