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

Unified Diff: remoting/webapp/crd/js/client_plugin_impl.js

Issue 825793002: Remove MediaStream renderer from the webapp. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 | « remoting/webapp/crd/js/client_plugin.js ('k') | remoting/webapp/crd/js/client_session.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/webapp/crd/js/client_plugin_impl.js
diff --git a/remoting/webapp/crd/js/client_plugin_impl.js b/remoting/webapp/crd/js/client_plugin_impl.js
index e0773d790383ecc595b14019dbf2850011383e41..73f7c17770f9ee8e5de51dbf3a1b3274b4fb45d8 100644
--- a/remoting/webapp/crd/js/client_plugin_impl.js
+++ b/remoting/webapp/crd/js/client_plugin_impl.js
@@ -120,12 +120,6 @@ remoting.ClientPluginImpl = function(container, onExtensionMessage,
this.onCastExtensionHandler_ = function(data) {};
/**
- * @type {remoting.MediaSourceRenderer}
- * @private
- */
- this.mediaSourceRenderer_ = null;
-
- /**
* @type {number}
* @private
*/
@@ -509,27 +503,6 @@ remoting.ClientPluginImpl.prototype.handleMessageMethod_ = function(message) {
break;
}
- } else if (message.method == 'mediaSourceReset') {
- if (!this.mediaSourceRenderer_) {
- console.error('Unexpected mediaSourceReset.');
- return;
- }
- this.mediaSourceRenderer_.reset(getStringAttr(message.data, 'format'))
-
- } else if (message.method == 'mediaSourceData') {
- if (!(message.data['buffer'] instanceof ArrayBuffer)) {
- console.error('Invalid mediaSourceData message:', message.data);
- return;
- }
- if (!this.mediaSourceRenderer_) {
- console.error('Unexpected mediaSourceData.');
- return;
- }
- // keyframe flag may be absent from the message.
- var keyframe = !!message.data['keyframe'];
- this.mediaSourceRenderer_.onIncomingData(
- (/** @type {ArrayBuffer} */ message.data['buffer']), keyframe);
-
} else if (message.method == 'unsetCursorShape') {
this.updateMouseCursorImage_('', 0, 0);
@@ -917,21 +890,6 @@ remoting.ClientPluginImpl.prototype.sendClientMessage =
};
-/**
- * Request MediaStream-based rendering.
- *
- * @param {remoting.MediaSourceRenderer} mediaSourceRenderer
- */
-remoting.ClientPluginImpl.prototype.enableMediaSourceRendering =
- function(mediaSourceRenderer) {
- if (!this.hasFeature(remoting.ClientPlugin.Feature.MEDIA_SOURCE_RENDERING)) {
- return;
- }
- this.mediaSourceRenderer_ = mediaSourceRenderer;
- this.plugin_.postMessage(JSON.stringify(
- { method: 'enableMediaSourceRendering', data: {} }));
-};
-
remoting.ClientPluginImpl.prototype.getDesktopWidth = function() {
return this.desktopWidth_;
}
« no previous file with comments | « remoting/webapp/crd/js/client_plugin.js ('k') | remoting/webapp/crd/js/client_session.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698