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

Side by Side 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: Fix a test failure Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 // This Polymer element shows information from media that is currently cast 5 // This Polymer element shows information from media that is currently cast
6 // to a device. 6 // to a device.
7 Polymer({ 7 Polymer({
8 is: 'route-details', 8 is: 'route-details',
9 9
10 properties: { 10 properties: {
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 /** 49 /**
50 * Whether the custom controller extensionview is ready to be shown. 50 * Whether the custom controller extensionview is ready to be shown.
51 * @type {boolean} 51 * @type {boolean}
52 */ 52 */
53 isExtensionViewReady: { 53 isExtensionViewReady: {
54 type: Boolean, 54 type: Boolean,
55 value: false, 55 value: false,
56 }, 56 },
57 57
58 /** 58 /**
59 * The timestamp for when the route details view was opened. We initialize
60 * the value in a function so that the value is set when the element is
61 * loaded, rather than at page load.
62 * @private {number}
63 */
64 openTime_: {
65 type: Number,
66 value: function() {
67 return Date.now();
68 },
69 },
70
71 /**
59 * The route to show. 72 * The route to show.
60 * @type {?media_router.Route|undefined} 73 * @type {?media_router.Route|undefined}
61 */ 74 */
62 route: { 75 route: {
63 type: Object, 76 type: Object,
64 observer: 'onRouteChange_', 77 observer: 'onRouteChange_',
65 }, 78 },
66 79
67 /** 80 /**
68 * The cast mode shown to the user. Initially set to auto mode. (See 81 * The cast mode shown to the user. Initially set to auto mode. (See
(...skipping 213 matching lines...) Expand 10 before | Expand all | Expand 10 after
282 this.fire('join-route-click', {route: this.route}); 295 this.fire('join-route-click', {route: this.route});
283 } else { 296 } else {
284 this.fire('change-route-source-click', { 297 this.fire('change-route-source-click', {
285 route: this.route, 298 route: this.route,
286 selectedCastMode: 299 selectedCastMode:
287 this.computeSelectedCastMode_(this.shownCastModeValue, this.sink) 300 this.computeSelectedCastMode_(this.shownCastModeValue, this.sink)
288 }); 301 });
289 } 302 }
290 }, 303 },
291 }); 304 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698