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

Side by Side Diff: tools/perf/measurements/rasterize_and_record_micro.py

Issue 225023031: Hack SkRecord in as an alternate recording mode in cc/resources/picture. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase to latest patches Created 6 years, 8 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 unified diff | Download patch
« cc/resources/picture.cc ('K') | « skia/skia_library.gypi ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import sys 5 import sys
6 import time 6 import time
7 7
8 from telemetry.core.util import TimeoutException 8 from telemetry.core.util import TimeoutException
9 from telemetry.page import page_measurement 9 from telemetry.page import page_measurement
10 from telemetry.page import page_test 10 from telemetry.page import page_test
(...skipping 87 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 raise page_measurement.MeasurementFailure( 98 raise page_measurement.MeasurementFailure(
99 'Failed to schedule rasterize_and_record_micro') 99 'Failed to schedule rasterize_and_record_micro')
100 100
101 tab.WaitForJavaScriptExpression( 101 tab.WaitForJavaScriptExpression(
102 'window.benchmark_results.done', self.options.timeout) 102 'window.benchmark_results.done', self.options.timeout)
103 103
104 data = tab.EvaluateJavaScript('window.benchmark_results.results') 104 data = tab.EvaluateJavaScript('window.benchmark_results.results')
105 105
106 pixels_recorded = data['pixels_recorded'] 106 pixels_recorded = data['pixels_recorded']
107 record_time = data['record_time_ms'] 107 record_time = data['record_time_ms']
108 record_time_sk_null_canvas = data['record_time_sk_null_canvas_ms']
109 record_time_painting_disabled = data['record_time_painting_disabled_ms']
110 record_time_skr = data['record_time_skr_ms']
108 pixels_rasterized = data['pixels_rasterized'] 111 pixels_rasterized = data['pixels_rasterized']
109 rasterize_time = data['rasterize_time_ms'] 112 rasterize_time = data['rasterize_time_ms']
110 113
111 results.Add('pixels_recorded', 'pixels', pixels_recorded) 114 results.Add('pixels_recorded', 'pixels', pixels_recorded)
112 results.Add('record_time', 'ms', record_time) 115 results.Add('record_time', 'ms', record_time)
116 results.Add('record_time_sk_null_canvas', 'ms', record_time_sk_null_canvas)
117 results.Add('record_time_painting_disabled', 'ms',
118 record_time_painting_disabled)
119 results.Add('record_time_skr', 'ms', record_time_skr)
113 results.Add('pixels_rasterized', 'pixels', pixels_rasterized) 120 results.Add('pixels_rasterized', 'pixels', pixels_rasterized)
114 results.Add('rasterize_time', 'ms', rasterize_time) 121 results.Add('rasterize_time', 'ms', rasterize_time)
115 122
116 # TODO(skyostil): Remove this temporary workaround when reference build has 123 # TODO(skyostil): Remove this temporary workaround when reference build has
117 # been updated to branch 1931 or later. 124 # been updated to branch 1931 or later.
118 if ((self._chrome_branch_number and self._chrome_branch_number >= 1931) or 125 if ((self._chrome_branch_number and self._chrome_branch_number >= 1931) or
119 sys.platform == 'android'): 126 sys.platform == 'android'):
120 record_time_sk_null_canvas = data['record_time_sk_null_canvas_ms'] 127 record_time_sk_null_canvas = data['record_time_sk_null_canvas_ms']
121 record_time_painting_disabled = data['record_time_painting_disabled_ms'] 128 record_time_painting_disabled = data['record_time_painting_disabled_ms']
122 results.Add('record_time_sk_null_canvas', 'ms', 129 results.Add('record_time_sk_null_canvas', 'ms',
(...skipping 16 matching lines...) Expand all
139 results.Add('pixels_rasterized_with_non_solid_color', 'pixels', 146 results.Add('pixels_rasterized_with_non_solid_color', 'pixels',
140 pixels_rasterized_with_non_solid_color) 147 pixels_rasterized_with_non_solid_color)
141 results.Add('pixels_rasterized_as_opaque', 'pixels', 148 results.Add('pixels_rasterized_as_opaque', 'pixels',
142 pixels_rasterized_as_opaque) 149 pixels_rasterized_as_opaque)
143 results.Add('total_layers', 'count', total_layers) 150 results.Add('total_layers', 'count', total_layers)
144 results.Add('total_picture_layers', 'count', total_picture_layers) 151 results.Add('total_picture_layers', 'count', total_picture_layers)
145 results.Add('total_picture_layers_with_no_content', 'count', 152 results.Add('total_picture_layers_with_no_content', 'count',
146 total_picture_layers_with_no_content) 153 total_picture_layers_with_no_content)
147 results.Add('total_picture_layers_off_screen', 'count', 154 results.Add('total_picture_layers_off_screen', 'count',
148 total_picture_layers_off_screen) 155 total_picture_layers_off_screen)
OLDNEW
« cc/resources/picture.cc ('K') | « skia/skia_library.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698