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

Issue 2170603004: Allow idle/suspend with 3+ monitors attached (Closed)

Created:
4 years, 5 months ago by dnicoara
Modified:
4 years, 5 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2785
Target Ref:
refs/pending/branch-heads/2785
Project:
chromium
Visibility:
Public.

Description

Allow idle/suspend with 3+ monitors attached On Samus it is possible to connect 2 external displays and on other chromebooks it is possible to connect 2 DisplayLink devices to have more than 2 monitors. In such cases turning off displays causes the display configuration to fail and present a "Dear monitor..." message even though it is possible to turn off these displays. BUG=627795 TEST=display_unittests and ran on Samus with 2 DisplayLink devices attached. Review-Url: https://codereview.chromium.org/2152303002 Cr-Commit-Position: refs/heads/master@{#406555} (cherry picked from commit 310889b982ca44844dbb79b409b874f3267c58fe) Committed: https://chromium.googlesource.com/chromium/src/+/44265ba65c8d3b5a59d910fa27cbd1454e5c5806

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+128 lines, -6 lines) Patch
M ui/display/chromeos/display_configurator.cc View 1 chunk +5 lines, -4 lines 0 comments Download
M ui/display/chromeos/display_configurator_unittest.cc View 3 chunks +122 lines, -1 line 0 comments Download
M ui/display/chromeos/update_display_configuration_task.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
dnicoara
4 years, 5 months ago (2016-07-21 15:33:17 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
44265ba65c8d3b5a59d910fa27cbd1454e5c5806.

Powered by Google App Engine
This is Rietveld 408576698