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

Unified Diff: build/android/surface_stats.py

Issue 18341006: SurfaceStats / Telemetry: calculate FPS in different frame buckets. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch Created 7 years, 6 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: build/android/surface_stats.py
diff --git a/build/android/surface_stats.py b/build/android/surface_stats.py
index 1bce6ef4ac424deee4ca947fc6953e48efefe6c4..0012e8ce88512e47c83ecda0daef13f04d8c98e4 100755
--- a/build/android/surface_stats.py
+++ b/build/android/surface_stats.py
@@ -20,9 +20,11 @@ from pylib.utils import run_tests_helper
_FIELD_FORMAT = {
'jank_count (janks)': '%d',
'max_frame_delay (vsyncs)': '%d',
- 'avg_surface_fps (fps)': '%.2f',
'frame_lengths (vsyncs)': '%.3f',
'refresh_period (seconds)': '%.6f',
+ 'avg_surface_fps (fps)': '%.2f',
+ 'avg_surface_fps_0.9 (fps)': '%.2f',
Sami 2013/07/03 15:11:50 I think we should just hide the windowed results i
+ 'avg_surface_fps_0.5 (fps)': '%.2f',
}
@@ -84,8 +86,9 @@ def main(argv):
parser.add_option('-f',
'--fields',
dest='fields',
- default='jank_count,max_frame_delay,avg_surface_fps,'
- 'frame_lengths',
+ default='jank_count,max_frame_delay,'
+ 'frame_lengths,avg_surface_fps,avg_surface_fps_0_9,'
+ 'avg_surface_fps_0_5,',
help='Comma separated list of fields to display or "all".')
parser.add_option('-d',
'--delay',

Powered by Google App Engine
This is Rietveld 408576698