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

Issue 2819933003: Report swap chain format histogram on every swap. (Closed)

Created:
3 years, 8 months ago by jbauman
Modified:
3 years, 8 months ago
Reviewers:
sunnyps
CC:
chromium-reviews, piman+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Report swap chain format histogram on every swap. It's more useful to know what fraction of presents are using BGRA vs. YUY2, rather than what fraction of swapchains, since some swapchains could be doing very few presents and wouldn't matter for performance. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2819933003 Cr-Commit-Position: refs/heads/master@{#464842} Committed: https://chromium.googlesource.com/chromium/src/+/6e8419d3411ff8f7629e9bf200ecdd8310f03ad9

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M gpu/ipc/service/direct_composition_surface_win.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 14 (10 generated)
jbauman
3 years, 8 months ago (2017-04-14 23:32:59 UTC) #5
sunnyps
lgtm
3 years, 8 months ago (2017-04-15 00:54:32 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2819933003/1
3 years, 8 months ago (2017-04-15 00:58:16 UTC) #11
commit-bot: I haz the power
3 years, 8 months ago (2017-04-15 01:04:56 UTC) #14
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/6e8419d3411ff8f7629e9bf200ec...

Powered by Google App Engine
This is Rietveld 408576698