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

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: Rebase 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 9ff13ec81a445e3097daf0d737ffe5046cae95b7..ef694695d45c701e7afd7702991aa0b2cb0c0dfb 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,12 +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);
@@ -298,6 +296,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