Commit 3046d581 authored by Derek Cheng's avatar Derek Cheng Committed by Commit Bot

[Media Router] Misc cleanup in media_router_bindings.js

Remove some backward compatibility logic that were put in more than a
few milestones ago.

Change-Id: I34c1959118b7c8fef8ae4e74df465f14af4db079
Reviewed-on: https://chromium-review.googlesource.com/580109
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Reviewed-by: default avatarTakumi Fujimoto <takumif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488517}
parent edb4808c
......@@ -89,11 +89,11 @@ define('media_router_bindings', [
'icon_url': route.iconUrl,
'is_local': route.isLocal,
'custom_controller_path': route.customControllerPath,
'for_display': route.forDisplay,
'is_incognito': route.offTheRecord,
'is_offscreen_presentation': route.isOffscreenPresentation,
// Begin newly added properties, followed by the milestone they were
// added. The guard should be safe to remove N+2 milestones later.
'for_display': route.forDisplay, // M47
'is_incognito': !!route.offTheRecord, // M50
'is_offscreen_presentation': !!route.isOffscreenPresentation, // M56
'supports_media_route_controller':
!!route.supportsMediaRouteController // M61
});
......@@ -619,10 +619,6 @@ define('media_router_bindings', [
* @param {!MediaRouterHandlers} handlers
*/
MediaRouteProvider.prototype.setHandlers = function(handlers) {
// TODO(mfoltz): Remove when component that supports this method is
// rolled out to all Chrome channels in M56.
if (!handlers['onBeforeInvokeHandler'])
handlers['onBeforeInvokeHandler'] = () => {};
this.handlers_ = handlers;
var requiredHandlers = [
'stopObservingMediaRoutes',
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment