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

Issue 1127233009: Perform a display configuration in software mirror on hotplug event (Closed)

Created:
5 years, 7 months ago by dnicoara
Modified:
5 years, 7 months ago
Reviewers:
CC:
chromium-reviews, stevenjb+watch_chromium.org, oshima+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@2403
Target Ref:
refs/pending/branch-heads/2403
Project:
chromium
Visibility:
Public.

Description

Perform a display configuration in software mirror on hotplug event On a hotplug event we configure the displays regardless of the state. However if we're in software mirror mode the configuration was skipped if the previous configuration was extended mode. We want to have a consistent approach to display configuration, so make sure we configure the displays if the configuration was triggered by a system event. Otherwise (if the configuration was requested by the user) we'll just skip the configuration so we don't perform unnecessary modesets. BUG=488736 Review URL: https://codereview.chromium.org/1133743005 Cr-Commit-Position: refs/heads/master@{#330534} (cherry picked from commit 1559e6b945b7c0d24c6d3f79a6c6dd811690fc4c) Committed: https://chromium.googlesource.com/chromium/src/+/423044fb9e9c0889f73ce329c017a13f5fa03351

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+105 lines, -2 lines) Patch
M ui/display/chromeos/update_display_configuration_task.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ui/display/chromeos/update_display_configuration_task_unittest.cc View 4 chunks +103 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
dnicoara
5 years, 7 months ago (2015-05-19 21:47:45 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
423044fb9e9c0889f73ce329c017a13f5fa03351.

Powered by Google App Engine
This is Rietveld 408576698