[MediaRouter UI] Compile MR WebUI js code with Closure.

chrome/browser/resources/media_router/media_router.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 <include src="media_router_data.js">
 <include src="media_router_ui_interface.js">
 
 // Handles user events for the Media Router UI.
 cr.define('media_router', function() {
   'use strict';
 
-  // The media-router-container element. Initialized after polymer is ready.
+  /**
+   * The media-router-container element. Initialized after polymer is ready.
+   * @type {?MediaRouterContainerElement}
+   */
   var container = null;
 
   /**
    * Initializes the Media Router WebUI and requests initial media
    * router content, such as the media sink and media route lists.
    */
   function initialize() {
     media_router.browserApi.requestInitialData();
 
-    container = $('media-router-container');
+    container = /** @type {!MediaRouterContainerElement} */
+        ($('media-router-container'));
     media_router.ui.setContainer(container);
 
     container.addEventListener('close-button-click', onCloseDialogEvent);
     container.addEventListener('close-dialog', onCloseDialogEvent);
     container.addEventListener('close-route-click', onCloseRouteClick);
     container.addEventListener('create-route', onCreateRoute);
     container.addEventListener('issue-action-click', onIssueActionClick);
     container.addEventListener('report-sink-count', onSinkCountReported);
   }
 
   /**
    * Closes the dialog.
    * Called when the user clicks the close button on the dialog.
    */
   function onCloseDialogEvent() {
     media_router.browserApi.closeDialog();
   }
 
   /**
    * Acts on an issue and dismisses it from the UI.
    * Called when the user performs an action on an issue.
    *
-   * @param {{detail: {id: string, actionType: number, helpPageId: number}}}
-   *     data
+   * @param {!Event} data
    * Parameters in |data|.detail:
-   *   id - issue ID.
-   *   actionType - type of action performed by the user.
-   *   helpPageId - the numeric help center ID.
+   *   id (string) - issue ID.
+   *   actionType (number) - type of action performed by the user.
+   *   helpPageId (number) - the numeric help center ID.

[Dan Beam, 2015/12/10 23:41:34]

this is not better than what you had before...

[imcheng, 2015/12/11 01:14:59]

Okay. But Event is not the same as {{detail: {id: string, actionType: number,
helpPageId: number}}} because it has other properties besides 'detail', which
causes compiler to complain. I tried to compensate for that by adding the types
here though they're worse than annotations.

    */
   function onIssueActionClick(data) {
     media_router.browserApi.actOnIssue(data.detail.id,
                                        data.detail.actionType,
                                        data.detail.helpPageId);
     container.issue = null;
   }
 
   /**
    * Creates a media route.
    * Called when the user requests to create a media route.
    *
-   * @param {{detail: {sinkId: string, selectedCastModeValue: number}}} data

[Dan Beam, 2015/12/10 23:41:34]

same question: why are you undoing this?

[imcheng, 2015/12/11 01:14:59]

See above

+   * @param {!Event} data
    * Parameters in |data|.detail:
-   *   sinkId - sink ID selected by the user.
-   *   selectedCastModeValue - cast mode selected by the user.
+   *   sinkId (string) - sink ID selected by the user.
+   *   selectedCastModeValue (number) - cast mode selected by the user.
    */
   function onCreateRoute(data) {
     media_router.browserApi.requestRoute(data.detail.sinkId,
                                          data.detail.selectedCastModeValue);
   }
 
   /**
    * Stops a route.
    * Called when the user requests to stop a media route.
    *
-   * @param {{detail: {route: string}}} data
+   * @param {!Event} data
    * Parameters in |data|.detail:
-   *   route - route ID.
+   *   route (media_router.Route) - The route to close.
    */
   function onCloseRouteClick(data) {
     media_router.browserApi.closeRoute(data.detail.route);
   }
 
   /**
    * Reports the current sink count.
    * Called 3 seconds after the dialog is initially opened.
    *
-   * @param {{detail: {sinkCount: number}}} data
+   * @param {!Event} data
    * Parameters in |data|.detail:
-   *   sinkCount - the number of sinks.
+   *   sinkCount (number) - the number of sinks.
    */
   function onSinkCountReported(data) {
     media_router.browserApi.reportSinkCount(data.detail.sinkCount);
   }
 
   return {
     initialize: initialize,
   };
 });
 
 window.addEventListener('load', media_router.initialize);