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

Unified Diff: tracing/tracing/metrics/system_health/webview_startup_metric.html

Issue 2162963002: [polymer] Merge of master into polymer10-migration (Closed) Base URL: git@github.com:catapult-project/catapult.git@polymer10-migration
Patch Set: Merge polymer10-migration int polymer10-merge Created 4 years, 5 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
« no previous file with comments | « tracing/tracing/metrics/system_health/utils.html ('k') | tracing/tracing/metrics/tracing_metric_test.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/metrics/system_health/webview_startup_metric.html
diff --git a/tracing/tracing/metrics/system_health/webview_startup_metric.html b/tracing/tracing/metrics/system_health/webview_startup_metric.html
index b10c9f44638edd7900d472d70a48d1c059c62da6..361ba0dec3179048577adbe9988486cbe07c544c 100644
--- a/tracing/tracing/metrics/system_health/webview_startup_metric.html
+++ b/tracing/tracing/metrics/system_health/webview_startup_metric.html
@@ -15,12 +15,12 @@ found in the LICENSE file.
tr.exportTo('tr.metrics.sh', function() {
function webviewStartupMetric(values, model) {
- model.iterateAllEvents(function(slice) {
+ for (var slice of model.getDescendantEvents()) {
// WebViewStartupInterval is the title of the section of code that is
// entered (via android.os.Trace.beginSection) when WebView is started
// up. This value is defined in TelemetryActivity.java.
if (!(slice instanceof tr.model.Slice))
- return;
+ continue;
if (slice.title === 'WebViewStartupInterval') {
values.addValue(new tr.v.NumericValue(
'webview_startup_wall_time',
@@ -52,7 +52,7 @@ tr.exportTo('tr.metrics.sh', function() {
slice.cpuDuration),
{ description: 'WebView blank URL load CPU time' }));
}
- });
+ }
};
tr.metrics.MetricRegistry.register(webviewStartupMetric);
« no previous file with comments | « tracing/tracing/metrics/system_health/utils.html ('k') | tracing/tracing/metrics/tracing_metric_test.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698