[Media Router] Add ProvideSinks() Mojo API

extensions/renderer/resources/media_router_bindings.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.
 
 var mediaRouter;
 
 define('media_router_bindings', [
     'content/public/renderer/frame_interfaces',
     'chrome/browser/media/router/mojo/media_router.mojom',
     'extensions/common/mojo/keep_alive.mojom',
     'mojo/common/time.mojom',
     'mojo/public/js/bindings',
     'url/mojo/origin.mojom',
+    'url/mojo/url.mojom',
 ], function(frameInterfaces,
             mediaRouterMojom,
             keepAliveMojom,
             timeMojom,
             bindings,
             originMojom) {
   'use strict';
 
   /**
    * Converts a media sink to a MediaSink Mojo object.
@@ skipping 490 matching lines @@
 
     /**
      * @type {function()}
      */
     this.updateMediaSinks = null;
 
     /**
      * @type {function(!string, !string, !SinkSearchCriteria): !string}
      */
     this.searchSinks = null;
+
+    /**
+     * @type {function()}
+     */
+    this.provideSinks = null;
   };
 
   /**
    * Routes calls from Media Router to the provider manager extension.
    * Registered with the MediaRouter stub.
    * @param {!MediaRouter} MediaRouter proxy to call into the
    * Media Router mojo interface.
    * @constructor
    */
   function MediaRouteProvider(mediaRouter) {
@@ skipping 31 matching lines @@
       'detachRoute',
       'terminateRoute',
       'joinRoute',
       'createRoute',
       'stopObservingMediaSinks',
       'startObservingMediaRoutes',
       'connectRouteByRouteId',
       'enableMdnsDiscovery',
       'updateMediaSinks',
       'searchSinks',
+      'provideSinks',
       'onBeforeInvokeHandler'
     ];
     requiredHandlers.forEach(function(nextHandler) {
       if (handlers[nextHandler] === undefined) {
         console.error(nextHandler + ' handler not registered.');
       }
     });
   }
 
   /**
@@ skipping 250 matching lines @@
     // to be missing this API.
     if (!this.handlers_.searchSinks) {
       return Promise.resolve({'sink_id': ''});
     }
     this.handlers_.onBeforeInvokeHandler();
     return Promise.resolve({
       'sink_id': this.handlers_.searchSinks(sinkId, sourceUrn, searchCriteria)
     });
   };
 
+  /**
+   * Notifies the provider manager that MediaRouter has discovered a list of
+   * sinks.
+   * @param {!Array<!mediaRouterMojom.MediaSink>} sinks
+   */
+  MediaRouteProvider.prototype.provideSinks = function(sinks) {

[btolsch, 2017/03/02 06:04:42]

This seems to be inconsistent with cl/146753004 and media_router.mojom (missing
providerName argument).

[zhaobin, 2017/03/02 20:24:04]

Done.

+    this.handlers_.onBeforeInvokeHandler();
+    this.handlers_.provideSinks(sinks);
+  };
+
   mediaRouter = new MediaRouter(new mediaRouterMojom.MediaRouterPtr(
       frameInterfaces.getInterface(mediaRouterMojom.MediaRouter.name)));
 
   return mediaRouter;
 });