Non-Local Join for Media Router and Presentation API

chrome/browser/resources/media_router/elements/route_details/route_details.js

Index: chrome/browser/resources/media_router/elements/route_details/route_details.js
diff --git a/chrome/browser/resources/media_router/elements/route_details/route_details.js b/chrome/browser/resources/media_router/elements/route_details/route_details.js
index 92ba82eccf98442b937c07126e479281c688eb5e..2b0f4ff99fdd210b0cbd1c09bf4ee5ebe90da012 100644
--- a/chrome/browser/resources/media_router/elements/route_details/route_details.js
+++ b/chrome/browser/resources/media_router/elements/route_details/route_details.js
@@ -47,6 +47,15 @@ Polymer({
     },
 
     /**
+     * The text for the join button.
+     * @private {string}
+     */
+    joinButtonText_: {
+      type: String,

[imcheng, 2015/12/01 23:45:06]

readOnly: true

[matt.boetger, 2015/12/03 01:19:20]

Done.

+      value: loadTimeData.getString('joinButton'),
+    },
+
+    /**
      * The text for the stop casting button.
      * @private {string}
      */
@@ -69,6 +78,16 @@ Polymer({
   },
 
   /**
+   * Fires a join-route-click event. This is called when the button to

[imcheng, 2015/12/01 23:45:06]

to join

[matt.boetger, 2015/12/03 01:19:21]

Done.

+   * the current route is clicked.
+   *
+   * @private
+   */
+  joinRoute_: function() {

[imcheng, 2015/12/01 23:45:06]

I wonder if this should somehow go through media-router-container. The reason I
say this is because we have some logic there that changes the sink being
launched to be a spinner and also prevent concurrent createRoute requests. It
would be nice to have that for join as well. But it depends on how the join flow
should look acoording to UX.

[matt.boetger, 2015/12/03 01:19:20]

Since all the route information is known at the time of the join, I don't think
there's any reason to have a spinner - we can immediately show all the necessary
information (and basically just hide the join button).  

+    this.fire('join-route-click', {route: this.route});
+  },
+
+  /**
    * Fires a close-route-click event. This is called when the button to close
    * the current route is clicked.
    *