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

Unified Diff: tools/perf/perf_tools/scrolling_benchmark.py

Issue 11028021: cc: Improve frame/commit accounting (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Make it work in single tread mode. Get rid of active concept. Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/perf/perf_tools/scrolling_benchmark.py
diff --git a/tools/perf/perf_tools/scrolling_benchmark.py b/tools/perf/perf_tools/scrolling_benchmark.py
index d472a2ac8efe0e016af4bc51594dc9862c29d1a7..5fc9f56cc10c5be56f81d10b075b455b6a9929f1 100644
--- a/tools/perf/perf_tools/scrolling_benchmark.py
+++ b/tools/perf/perf_tools/scrolling_benchmark.py
@@ -11,7 +11,7 @@ class DidNotScrollException(multi_page_benchmark.MeasurementFailure):
super(DidNotScrollException, self).__init__('Page did not scroll')
def CalcScrollResults(rendering_stats_deltas, results):
- num_frames_sent_to_screen = rendering_stats_deltas['numFramesSentToScreen']
+ num_frames_sent_to_screen = rendering_stats_deltas['rendererFrameCount']
mean_frame_time_seconds = (
rendering_stats_deltas['totalTimeInSeconds'] /
@@ -19,7 +19,7 @@ def CalcScrollResults(rendering_stats_deltas, results):
dropped_percent = (
rendering_stats_deltas['droppedFrameCount'] /
- float(num_frames_sent_to_screen))
+ float(rendering_stats_deltas['vsyncCount']))
results.Add('mean_frame_time', 'ms', round(mean_frame_time_seconds * 1000, 3))
results.Add('dropped_percent', '%', round(dropped_percent * 100, 1))
@@ -69,7 +69,7 @@ class ScrollingBenchmark(multi_page_benchmark.MultiPageBenchmark):
rendering_stats_deltas = tab.runtime.Evaluate(
'window.__renderingStatsDeltas')
- if not (rendering_stats_deltas['numFramesSentToScreen'] > 0):
+ if not (rendering_stats_deltas['rendererFrameCount'] > 0):
raise DidNotScrollException()
return rendering_stats_deltas

Powered by Google App Engine
This is Rietveld 408576698