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

Unified Diff: chrome/browser/resources/media_router/elements/route_details/route_details.js

Issue 2915983002: [Media Router] Add UMA metrics to record the load time for route controls (Closed)
Patch Set: . Created 3 years, 7 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: chrome/browser/resources/media_router/elements/route_details/route_details.js
diff --git a/chrome/browser/resources/media_router/elements/route_details/route_details.js b/chrome/browser/resources/media_router/elements/route_details/route_details.js
index da76f91294a601e8d132669e91af44515fa4b230..ddf8d76fd5ce715d72a86196beca6f12afb74161 100644
--- a/chrome/browser/resources/media_router/elements/route_details/route_details.js
+++ b/chrome/browser/resources/media_router/elements/route_details/route_details.js
@@ -56,6 +56,17 @@ Polymer({
},
/**
+ * The time when the route details view was opened. We do not set this value
+ * in onOpened() because that may not get called early enough if this view
+ * hasn't previously been loaded.
+ * @type {?Date}
+ */
+ openTime: {
+ type: Date,
+ value: null,
+ },
+
+ /**
* The route to show.
* @type {?media_router.Route|undefined}
*/

Powered by Google App Engine
This is Rietveld 408576698