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

Issue 1038263002: [Ozone] Allow surfaceless path to have 2 frames pending (Closed)

Created:
5 years, 9 months ago by dnicoara
Modified:
5 years, 9 months ago
Reviewers:
danakj, brianderson
CC:
chromium-reviews, jam, sievers+watch_chromium.org, jbauman+watch_chromium.org, darin-cc_chromium.org, kalyank, piman+watch_chromium.org, danakj+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Ozone] Allow surfaceless path to have 2 frames pending Allows the surfaceless path (used for Ozone/Freon) to queue up more work while waiting for the currently displayed buffer to be released by the hardware. In turn this means slightly better frame rates on WebGL apps as we're able to better hit the vsync deadline. BUG=467135 TESTS=Ran on stout with Freon and verified that the FPS on aquarium increased to similar values as X11 Committed: https://crrev.com/9df811e63e97ca7c079a0eb13ad8814716ad123d Cr-Commit-Position: refs/heads/master@{#322634}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.cc View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (2 generated)
dnicoara
danakj@ for OWNERS review brianderson@ FYI
5 years, 9 months ago (2015-03-27 17:40:00 UTC) #2
danakj
LGTM % brianderson
5 years, 9 months ago (2015-03-27 17:40:53 UTC) #3
brianderson
lgtm
5 years, 9 months ago (2015-03-27 18:53:37 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1038263002/1
5 years, 9 months ago (2015-03-27 20:00:41 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 9 months ago (2015-03-27 20:30:42 UTC) #7
commit-bot: I haz the power
5 years, 9 months ago (2015-03-27 20:31:21 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/9df811e63e97ca7c079a0eb13ad8814716ad123d
Cr-Commit-Position: refs/heads/master@{#322634}

Powered by Google App Engine
This is Rietveld 408576698