diff options
author | ernstm@chromium.org <ernstm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-19 18:14:24 +0000 |
---|---|---|
committer | ernstm@chromium.org <ernstm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-10-19 18:14:24 +0000 |
commit | 9f123a0fbd7d95a24a37b5e5069ee010124d3152 (patch) | |
tree | dfe24d77c5f042ddc872a4166931be8e1e97ddee /tools/perf/metrics/rendering_stats.py | |
parent | 22a1405d7ae953c97a4ea1146cc5297c9f9385e5 (diff) | |
download | chromium_src-9f123a0fbd7d95a24a37b5e5069ee010124d3152.zip chromium_src-9f123a0fbd7d95a24a37b5e5069ee010124d3152.tar.gz chromium_src-9f123a0fbd7d95a24a37b5e5069ee010124d3152.tar.bz2 |
telemetry: Smoothness metric clean-up.
This patch removes compatibility with none-Chrome browsers and generally
cleans up metrics/smoothness.py
R=qyearsley@chromium.org,dominikg@chromium.org,tonyg@chromium.org
BUG=305347
Review URL: https://codereview.chromium.org/27165005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229602 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/perf/metrics/rendering_stats.py')
-rw-r--r-- | tools/perf/metrics/rendering_stats.py | 27 |
1 files changed, 9 insertions, 18 deletions
diff --git a/tools/perf/metrics/rendering_stats.py b/tools/perf/metrics/rendering_stats.py index da1713d..9bde896 100644 --- a/tools/perf/metrics/rendering_stats.py +++ b/tools/perf/metrics/rendering_stats.py @@ -4,9 +4,11 @@ from operator import attrgetter +RENDER_PROCESS_MARKER = 'RenderProcessMarker' + + class RenderingStats(object): - def __init__(self, render_process_marker, timeline_markers, - rendering_stats_deltas, used_gpu_benchmarking): + def __init__(self, render_process_marker, timeline_markers): """ Utility class for extracting rendering statistics from the timeline (or other loggin facilities), and providing them in a common format to classes @@ -34,22 +36,11 @@ class RenderingStats(object): self.rasterize_time = [] self.rasterized_pixel_count = [] - if used_gpu_benchmarking: - for marker in timeline_markers: - self.initMainThreadStatsFromTimeline(marker.start, - marker.start+marker.duration) - self.initImplThreadStatsFromTimeline(marker.start, - marker.start+marker.duration) - else: - self.initFrameCountsFromRenderingStats(rendering_stats_deltas) - - def initFrameCountsFromRenderingStats(self, rs): - # TODO(ernstm): remove numFramesSentToScreen when RenderingStats - # cleanup CL was picked up by the reference build. - if 'frameCount' in rs: - self.frame_count = rs.get('frameCount', 0) - else: - self.frame_count = rs.get('numFramesSentToScreen', 0) + for marker in timeline_markers: + self.initMainThreadStatsFromTimeline(marker.start, + marker.start+marker.duration) + self.initImplThreadStatsFromTimeline(marker.start, + marker.start+marker.duration) def initMainThreadStatsFromTimeline(self, start, end): # TODO(ernstm): Remove when CL with new event names was rolled into |