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

Issue 2954343003: Sets the rotation source as USER when using the system display API (Closed)

Created:
3 years, 5 months ago by malaykeshav
Modified:
3 years, 5 months ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/branch-heads/3112
Project:
chromium
Visibility:
Public.

Description

Sets the rotation source as USER when using the system display API The rotation preference is not persistent across reboot since we only store the preference if the rotation source is 'USER'. With the move to MD settings, we now use the System Display API to update any display settings, which still uses the source of rotation as ACTIVE. This change updates the source of rotation to USER. We persist the setting even if the change has been made by an application. (This is consistent with the other settings that are being set via this API.) BUG=716110 COMPONENT=Display Info Provider, Chrome Settings Review-Url: https://codereview.chromium.org/2920703003 Cr-Original-Commit-Position: refs/heads/master@{#476734} Review-Url: https://codereview.chromium.org/2954343003 . Cr-Commit-Position: refs/branch-heads/3112@{#474} Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897} Committed: https://chromium.googlesource.com/chromium/src/+/c6d6bd1be3e6042c5022afdb40fc5334ed0bc7e0

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M chrome/browser/extensions/display_info_provider_chromeos.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
malaykeshav
3 years, 5 months ago (2017-06-27 01:26:31 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
c6d6bd1be3e6042c5022afdb40fc5334ed0bc7e0 (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698