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

Unified Diff: extensions/renderer/resources/media_router_bindings.js

Issue 1507743005: [MediaRouter] Renames CloseRoute() to Terminate() and creates DetachRoute() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix android build Created 5 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/renderer/presentation/presentation_dispatcher.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 7bf09d3bf9ab598f5fdd89024c20877802727f60..462fd2fa7ade565a1fcc44809d4d78f463a36580 100644
--- a/extensions/renderer/resources/media_router_bindings.js
+++ b/extensions/renderer/resources/media_router_bindings.js
@@ -326,7 +326,7 @@ define('media_router_bindings', [
/**
* @type {function(string)}
*/
- this.closeRoute = null;
+ this.terminateRoute = null;
/**
* @type {function(string)}
@@ -362,7 +362,7 @@ define('media_router_bindings', [
/**
* @type {function(string)}
*/
- this.onPresentationSessionDetached = null;
+ this.detachRoute = null;
/**
* @type {function()}
@@ -414,8 +414,8 @@ define('media_router_bindings', [
'sendRouteBinaryMessage',
'listenForRouteMessages',
'stopListeningForRouteMessages',
- 'onPresentationSessionDetached',
- 'closeRoute',
+ 'detachRoute',
+ 'terminateRoute',
'joinRoute',
'createRoute',
'stopObservingMediaSinks',
@@ -498,11 +498,11 @@ define('media_router_bindings', [
};
/**
- * Closes the route specified by |routeId|.
+ * Terminates the route specified by |routeId|.
* @param {!string} routeId
*/
- MediaRouteProvider.prototype.closeRoute = function(routeId) {
- this.handlers_.closeRoute(routeId);
+ MediaRouteProvider.prototype.terminateRoute = function(routeId) {
+ this.handlers_.terminateRoute(routeId);
};
/**
@@ -566,13 +566,13 @@ define('media_router_bindings', [
};
/**
- * Indicates that the presentation session that was connected to |routeId| is
- * no longer connected to it.
+ * Indicates that the presentation connection that was connected to |routeId|
+ * is no longer connected to it.
* @param {!string} routeId
*/
- MediaRouteProvider.prototype.onPresentationSessionDetached = function(
+ MediaRouteProvider.prototype.detachRoute = function(
routeId) {
- this.handlers_.onPresentationSessionDetached(routeId);
+ this.handlers_.detachRoute(routeId);
};
/**
« no previous file with comments | « content/renderer/presentation/presentation_dispatcher.cc ('k') | tools/metrics/histograms/histograms.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698