Pass origin to StartObservingMediaSinks.

extensions/renderer/resources/media_router_bindings.js

Index: extensions/renderer/resources/media_router_bindings.js
diff --git a/extensions/renderer/resources/media_router_bindings.js b/extensions/renderer/resources/media_router_bindings.js
index b6252c50e999fe49cbc08973532b30b1db95939b..7178956c2c6ec053d88f8c10f990352949a7a032 100644
--- a/extensions/renderer/resources/media_router_bindings.js
+++ b/extensions/renderer/resources/media_router_bindings.js
@@ -264,10 +264,12 @@ define('media_router_bindings', [
    * Called by the provider manager when a sink list for a given source is
    * updated.
    * @param {!string} sourceUrn
+   * @param {!string} origin
    * @param {!Array<!MediaSink>} sinks
    */
-  MediaRouter.prototype.onSinksReceived = function(sourceUrn, sinks) {
-    this.service_.onSinksReceived(sourceUrn, sinks.map(sinkToMojo_));
+  MediaRouter.prototype.onSinksReceived = function(
+      sourceUrn, origin, sinks) {
+    this.service_.onSinksReceived(sourceUrn, origin, sinks.map(sinkToMojo_));

[mark a. foltz, 2016/03/03 22:58:37]

How will this handle an extension that does not use the new API?

Rather than sticking an optional argument on the end, I would use a type check
on the second argument to determine which API is in use.

[matt.boetger, 2016/03/04 00:22:11]

Done.

   };
 
   /**
@@ -530,19 +532,21 @@ define('media_router_bindings', [
    * designated by |sourceUrn|.  Results are returned by calling
    * OnSinksReceived.
    * @param {!string} sourceUrn
+   * @param {!string} origin
    */
   MediaRouteProvider.prototype.startObservingMediaSinks =
-      function(sourceUrn) {
-    this.handlers_.startObservingMediaSinks(sourceUrn);
+      function(sourceUrn, origin) {
+    this.handlers_.startObservingMediaSinks(sourceUrn, origin);
   };
 
   /**
    * Stops querying for sinks capable of displaying |sourceUrn|.
    * @param {!string} sourceUrn
+   * @param {!string} origin
    */
   MediaRouteProvider.prototype.stopObservingMediaSinks =
-      function(sourceUrn) {
-    this.handlers_.stopObservingMediaSinks(sourceUrn);
+      function(sourceUrn, origin) {
+    this.handlers_.stopObservingMediaSinks(sourceUrn, origin);
   };
 
   /**