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

Issue 845383002: [Ozone-DRI] Make sure the display configuration is up-to-date on the GPU (Closed)

Created:
5 years, 11 months ago by dnicoara
Modified:
5 years, 11 months ago
CC:
chromium-reviews, kalyank, ozone-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Ozone-DRI] Make sure the display configuration is up-to-date on the GPU When probing for new displays we create new snapshots. We need to make sure that existing ones will contain the same state as the old ones otherwise the state between the Browser and GPU process will be out of sync. BUG=434115 NOTRY=true Committed: https://crrev.com/ffd7134b2bf1cd6909eba9a44836a2c719fb564b Cr-Commit-Position: refs/heads/master@{#311510}

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -13 lines) Patch
M ui/ozone/platform/dri/display_snapshot_dri.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M ui/ozone/platform/dri/dri_gpu_platform_support.cc View 1 chunk +7 lines, -2 lines 0 comments Download
M ui/ozone/platform/dri/native_display_delegate_dri.cc View 1 2 chunks +21 lines, -9 lines 0 comments Download

Messages

Total messages: 14 (5 generated)
dnicoara
PTAL, this should fix the software mirror mode issues.
5 years, 11 months ago (2015-01-13 17:02:34 UTC) #2
achaulk
lgtm
5 years, 11 months ago (2015-01-14 16:03:57 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/845383002/20001
5 years, 11 months ago (2015-01-14 17:47:55 UTC) #5
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
5 years, 11 months ago (2015-01-14 17:47:57 UTC) #7
dnicoara
+alexst@
5 years, 11 months ago (2015-01-14 17:48:54 UTC) #9
alexst (slow to review)
rubberstamp lgtm
5 years, 11 months ago (2015-01-14 18:05:39 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/845383002/20001
5 years, 11 months ago (2015-01-14 18:06:41 UTC) #12
commit-bot: I haz the power
Committed patchset #2 (id:20001)
5 years, 11 months ago (2015-01-14 18:08:01 UTC) #13
commit-bot: I haz the power
5 years, 11 months ago (2015-01-14 18:09:43 UTC) #14
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/ffd7134b2bf1cd6909eba9a44836a2c719fb564b
Cr-Commit-Position: refs/heads/master@{#311510}

Powered by Google App Engine
This is Rietveld 408576698