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

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: . 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 time when the route details view was opened. We do not set this value
60 * in onOpened() because that may not get called early enough if this view
61 * hasn't previously been loaded.
62 * @type {?Date}
63 */
64 openTime: {
65 type: Date,
66 value: null,
67 },
68
69 /**
59 * The route to show. 70 * The route to show.
60 * @type {?media_router.Route|undefined} 71 * @type {?media_router.Route|undefined}
61 */ 72 */
62 route: { 73 route: {
63 type: Object, 74 type: Object,
64 observer: 'onRouteChange_', 75 observer: 'onRouteChange_',
65 }, 76 },
66 77
67 /** 78 /**
68 * The cast mode shown to the user. Initially set to auto mode. (See 79 * 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}); 293 this.fire('join-route-click', {route: this.route});
283 } else { 294 } else {
284 this.fire('change-route-source-click', { 295 this.fire('change-route-source-click', {
285 route: this.route, 296 route: this.route,
286 selectedCastMode: 297 selectedCastMode:
287 this.computeSelectedCastMode_(this.shownCastModeValue, this.sink) 298 this.computeSelectedCastMode_(this.shownCastModeValue, this.sink)
288 }); 299 });
289 } 300 }
290 }, 301 },
291 }); 302 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698