Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(8)

Issue 11198005: NOT READY FOR REVIEW - switch to a subscriber model for rendering stats (Closed)

Created:
7 years ago by Ian Vollick
Modified:
6 years, 4 months ago
Reviewers:
CC:
chromium-reviews, cc-bugs_chromium.org
Visibility:
Public.

Description

NOT READY FOR REVIEW - switch to a subscriber model for rendering stats Currently, we use CCLTH::renderingStats(...) to collect our stats. It would make more sense if we were to subscribe for a period of time and collect the stats afterwards. BUG=155136

Patch Set 1 #

Patch Set 2 : Rendering stats now know which properies were modified. #

Patch Set 3 : . #

Patch Set 4 : Added some guts to the subscriber. #

Patch Set 5 : Rebased (most of the work done by hartmanng -- thanks, Glenn!) #

Patch Set 6 : Added missing file. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+282 lines, -14 lines) Patch
M cc/cc.gyp View 1 2 3 4 2 chunks +3 lines, -0 lines 0 comments Download
M cc/layer_tree_host.h View 1 2 3 4 3 chunks +8 lines, -0 lines 0 comments Download
M cc/layer_tree_host.cc View 1 2 3 4 2 chunks +14 lines, -0 lines 0 comments Download
M cc/layer_tree_host_impl.h View 1 2 3 4 3 chunks +4 lines, -0 lines 0 comments Download
M cc/layer_tree_host_impl.cc View 1 2 3 4 3 chunks +25 lines, -1 line 0 comments Download
M cc/proxy.h View 1 2 3 4 1 chunk +2 lines, -0 lines 0 comments Download
M cc/rendering_stats.h View 1 2 3 4 2 chunks +4 lines, -13 lines 0 comments Download
A cc/rendering_stats.cc View 1 2 3 4 5 1 chunk +25 lines, -0 lines 0 comments Download
A cc/rendering_stats_subscriber.h View 1 2 3 1 chunk +40 lines, -0 lines 0 comments Download
A cc/rendering_stats_subscriber.cc View 1 2 3 1 chunk +85 lines, -0 lines 0 comments Download
M cc/single_thread_proxy.h View 1 2 3 4 3 chunks +4 lines, -0 lines 0 comments Download
M cc/single_thread_proxy.cc View 1 2 3 4 2 chunks +18 lines, -0 lines 0 comments Download
M cc/thread_proxy.h View 1 2 3 4 4 chunks +7 lines, -0 lines 0 comments Download
M cc/thread_proxy.cc View 1 2 3 4 3 chunks +43 lines, -0 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698