[Media Router] Add a supports_web_ui_controller attribute to MediaRoute

chrome/browser/resources/media_router/elements/route_details/route_details.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.
 
 // This Polymer element shows information from media that is currently cast
 // to a device.
 Polymer({
   is: 'route-details',
 
   properties: {
@@ skipping 15 matching lines @@
       computed: 'computeChangeRouteSourceAvailable_(route, sink,' +
           'isAnySinkCurrentlyLaunching, shownCastModeValue)',
     },
 
     /**
      * An enum value to represent the controller to show.
      * @private {number}
      */
     controllerType_: {
       type: Number,
-      computed: 'computeControllerType_(useWebUiRouteControls,' +
-          'isExtensionViewReady)',
+      computed: 'computeControllerType_(route, isExtensionViewReady)',
     },
 
     /**
      * Whether a sink is currently launching in the container.
      * @type {boolean}
      */
     isAnySinkCurrentlyLaunching: {
       type: Boolean,
       value: false,
     },
@@ skipping 40 matching lines @@
     },
 
     /**
      * Sink associated with |route|.
      * @type {?media_router.Sink}
      */
     sink: {
       type: Object,
       value: null,
     },
-
-    /**
-     * Whether we should use the WebUI route controls. This value is used for
-     * updating |controllerType_|,
-     * @type {boolean}
-     */
-    useWebUiRouteControls: {
-      type: Boolean,
-      value: false,
-    },
   },
 
   behaviors: [
     I18nBehavior,
   ],
 
   /**
    * Fires a close-route event. This is called when the button to close
    * the current route is clicked.
    *
@@ skipping 36 matching lines @@
     if (!route.currentCastMode) {
       return true;
     }
     var selectedCastMode =
         this.computeSelectedCastMode_(shownCastModeValue, sink);
     return (selectedCastMode != 0) &&
         (selectedCastMode != route.currentCastMode);
   },
 
   /**
-   * @param {boolean} useWebUiRouteControls
+   * @param {?media_router.Route} route
    * @param {boolean} isExtensionViewReady
    * @return {number} An enum value to represent the controller to show.
    * @private
    */
-  computeControllerType_: function(
-      useWebUiRouteControls, isExtensionViewReady) {
-    if (useWebUiRouteControls) {
+  computeControllerType_: function(route, isExtensionViewReady) {
+    if (route && route.supportsWebUiController) {
       return media_router.ControllerType.WEBUI;
     }
     if (isExtensionViewReady) {
       return media_router.ControllerType.EXTENSION;
     }
     return media_router.ControllerType.NONE;
   },
 
   /**
    * @param {number} castMode User selected cast mode or AUTO.
@@ skipping 63 matching lines @@
       this.updateActivityStatus_();
     }
     if (newRoute &&
         this.controllerType_ === media_router.ControllerType.WEBUI &&
         this.$$('route-controls')) {
       this.$$('route-controls').onRouteUpdated(newRoute);
     }
   },
 
   /**
+   * @param {?media_router.Route} route
+   * @return {boolean}
+   * @private
+   */
+  shouldAttemptLoadingExtensionView_: function(route) {
+    return !!route && !route.supportsWebUiController;

[takumif, 2017/06/19 18:10:28]

The Closure compiler complains that the return type is boolean|null without
this.

[imcheng, 2017/06/19 19:56:13]

Acknowledged.

+  },
+
+  /**
    * @param {number} controllerType
    * @return {boolean} Whether the extensionview should be shown instead of the
    *     default route info element or the WebUI route controller.
    * @private
    */
   shouldShowExtensionView_: function(controllerType) {
     return controllerType === media_router.ControllerType.EXTENSION;
   },
 
   /**
@@ skipping 30 matching lines @@
       this.fire('join-route-click', {route: this.route});
     } else {
       this.fire('change-route-source-click', {
         route: this.route,
         selectedCastMode:
             this.computeSelectedCastMode_(this.shownCastModeValue, this.sink)
       });
     }
   },
 });