[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

[Dan Beam, 2015/12/12 01:43:02]

i would change this back to what it was because you're using this information as
data, not as an event

[imcheng, 2015/12/14 22:47:00]

This function expects an Event. What I had before was not correct:

## .../media_router.js:33: ERROR - actual parameter 2 of
Node.prototype.addEventListener does not match formal parameter
## found   : function ({detail: {actionType: number, helpPageId: number, id:
string}}): undefined
## required: (EventListener|function (Event): (boolean|undefined)|null)
##     container.addEventListener('issue-action-click', onIssueActionClick);

    * 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.
    */
   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
+   * @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);