Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(170)

Side by Side Diff: chrome/renderer/resources/extensions/media_router_bindings.js

Issue 2837363002: [Media Router] Use DialMediaSinkService in MediaRouterMojoImpl (Closed)
Patch Set: resolve code review comments from Mark Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 var mediaRouter; 5 var mediaRouter;
6 6
7 define('media_router_bindings', [ 7 define('media_router_bindings', [
8 'chrome/common/media_router/mojo/media_controller.mojom', 8 'chrome/common/media_router/mojo/media_controller.mojom',
9 'chrome/common/media_router/mojo/media_router.mojom', 9 'chrome/common/media_router/mojo/media_router.mojom',
10 'chrome/common/media_router/mojo/media_status.mojom', 10 'chrome/common/media_router/mojo/media_status.mojom',
(...skipping 276 matching lines...) Expand 10 before | Expand all | Expand 10 after
287 MediaStatusObserverPtr: mediaStatusMojom.MediaStatusObserverPtr, 287 MediaStatusObserverPtr: mediaStatusMojom.MediaStatusObserverPtr,
288 Sink: mediaRouterMojom.MediaSink, 288 Sink: mediaRouterMojom.MediaSink,
289 SinkExtraData: mediaRouterMojom.MediaSinkExtraData, 289 SinkExtraData: mediaRouterMojom.MediaSinkExtraData,
290 TimeDelta: timeMojom.TimeDelta, 290 TimeDelta: timeMojom.TimeDelta,
291 Url: urlMojom.Url, 291 Url: urlMojom.Url,
292 }; 292 };
293 }; 293 };
294 294
295 /** 295 /**
296 * Registers the Media Router Provider Manager with the Media Router. 296 * Registers the Media Router Provider Manager with the Media Router.
297 * @return {!Promise<string>} Instance ID for the Media Router. 297 * @return {!Promise<Object>} Instance ID and config for the Media Router.
298 */ 298 */
299 MediaRouter.prototype.start = function() { 299 MediaRouter.prototype.start = function() {
300 return this.service_.registerMediaRouteProvider( 300 return this.service_.registerMediaRouteProvider(
301 this.mediaRouteProviderBinding_.createInterfacePtrAndBind()).then( 301 this.mediaRouteProviderBinding_.createInterfacePtrAndBind());
302 function(result) {
303 return result.instance_id;
304 }.bind(this));
305 } 302 }
306 303
307 /** 304 /**
308 * Sets the service delegate methods. 305 * Sets the service delegate methods.
309 * @param {Object} handlers 306 * @param {Object} handlers
310 */ 307 */
311 MediaRouter.prototype.setHandlers = function(handlers) { 308 MediaRouter.prototype.setHandlers = function(handlers) {
312 this.mrpm_.setHandlers(handlers); 309 this.mrpm_.setHandlers(handlers);
313 } 310 }
314 311
(...skipping 638 matching lines...) Expand 10 before | Expand all | Expand 10 after
953 } 950 }
954 this.handlers_.onBeforeInvokeHandler(); 951 this.handlers_.onBeforeInvokeHandler();
955 this.handlers_.setMediaRouteStatusObserver(routeId, observer); 952 this.handlers_.setMediaRouteStatusObserver(routeId, observer);
956 }; 953 };
957 954
958 mediaRouter = new MediaRouter(new mediaRouterMojom.MediaRouterPtr( 955 mediaRouter = new MediaRouter(new mediaRouterMojom.MediaRouterPtr(
959 frameInterfaces.getInterface(mediaRouterMojom.MediaRouter.name))); 956 frameInterfaces.getInterface(mediaRouterMojom.MediaRouter.name)));
960 957
961 return mediaRouter; 958 return mediaRouter;
962 }); 959 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698