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

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

Issue 2938173004: [Media Router] Add a supports_web_ui_controller attribute to MediaRoute (Closed)
Patch Set: Make Closure happy 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/media_router/media_router.js
diff --git a/chrome/browser/resources/media_router/media_router.js b/chrome/browser/resources/media_router/media_router.js
index cd65124a6f4c75e43e7e895b8aaea7e38b4f12f7..41b60db13d4260d353e0ca08ba47e8a86117ce5f 100644
--- a/chrome/browser/resources/media_router/media_router.js
+++ b/chrome/browser/resources/media_router/media_router.js
@@ -12,7 +12,7 @@ cr.define('media_router', function() {
/**
* The media-router-container element. Initialized after polymer is ready.
- * @type {?MediaRouterContainerElement}
+ * @type {?MediaRouterContainerInterface}
*/
var container = null;
@@ -25,11 +25,10 @@ cr.define('media_router', function() {
if (!cr.isMac)
onRequestInitialData();
- container = /** @type {!MediaRouterContainerElement} */
+ container = /** @type {!MediaRouterContainerInterface} */
($('media-router-container'));
- media_router.ui.setElements(container,
- /** @type {!MediaRouterHeaderElement} */(container.header));
+ media_router.ui.setElements(container, container.header);
container.addEventListener('acknowledge-first-run-flow',
onAcknowledgeFirstRunFlow);
@@ -299,6 +298,7 @@ cr.define('media_router', function() {
* success - whether or not the route creation was successful.
*/
function onReportRouteCreation(event) {
+ /** @type {{success: boolean}} */
var detail = event.detail;
media_router.browserApi.reportRouteCreation(detail.success);
}

Powered by Google App Engine
This is Rietveld 408576698