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

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

Issue 855553003: Enable PepperVideoDecoder3D (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | no next file » | 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 17b4adf8aff4bfef555250bc3825deade1d339b4..af0fc723fbf5f618d6139be682443333135b71c3 100644
--- a/remoting/webapp/crd/js/client_plugin_impl.js
+++ b/remoting/webapp/crd/js/client_plugin_impl.js
@@ -648,6 +648,12 @@ remoting.ClientPluginImpl.prototype.connect = function(
} else if (remoting.platformIsChromeOS()) {
keyFilter = 'cros';
}
+ // Use PPB_VideoDecoder API only in Chrome 42 and above. It is broken in
+ // previous versions of Chrome, see http://crbug.com/447403 .
+ // Currently PPAPI doesn't provide a way for plugins to check the Chrome
+ // version, so this check needs to be in the webapp.
+ var enableVideoDecodeRenderer =
+ parseInt((remoting.getChromeVersion() || '0').split('.')[0], 10) >= 42;
this.plugin_.postMessage(JSON.stringify(
{ method: 'delegateLargeCursors', data: {} }));
this.plugin_.postMessage(JSON.stringify(
@@ -661,7 +667,8 @@ remoting.ClientPluginImpl.prototype.connect = function(
capabilities: this.capabilities_.join(" "),
clientPairingId: clientPairingId,
clientPairedSecret: clientPairedSecret,
- keyFilter: keyFilter
+ keyFilter: keyFilter,
+ enableVideoDecodeRenderer: enableVideoDecodeRenderer
}
}));
};
« no previous file with comments | « remoting/client/plugin/chromoting_instance.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698