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

Unified Diff: webkit/glue/plugins/plugin_host.cc

Issue 1569005: Fix CoreGraphics and Core Animation drawing for QuickTime plugin (Closed)
Patch Set: Spacing fix Created 10 years, 9 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 | « no previous file | webkit/glue/plugins/webplugin_delegate_impl_mac.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/plugins/plugin_host.cc
diff --git a/webkit/glue/plugins/plugin_host.cc b/webkit/glue/plugins/plugin_host.cc
index 5ceea68c8d06b104b67befb757a96f255ef618e0..b262188f17428c46ffc681630a8f5d1658c01fce 100644
--- a/webkit/glue/plugins/plugin_host.cc
+++ b/webkit/glue/plugins/plugin_host.cc
@@ -794,16 +794,7 @@ NPError NPN_GetValue(NPP id, NPNVariable variable, void* value) {
case NPNVsupportsCoreAnimationBool: {
// We only support the Core Animation model on 10.6 and higher
NPBool* supports_model = reinterpret_cast<NPBool*>(value);
- // Our Core Animation support currently doesn't handle QuickTime correctly
- // (see <http://crbug.com/38336>), so for now we don't allow QuickTime to
- // negotiate that model.
- scoped_refptr<NPAPI::PluginInstance> plugin = FindInstance(id);
- if (plugin &&
- plugin->plugin_lib()->plugin_info().name.find(L"QuickTime") !=
- std::wstring::npos)
- *supports_model = FALSE;
- else
- *supports_model = SupportsSharingAcceleratedSurfaces() ? TRUE : FALSE;
+ *supports_model = SupportsSharingAcceleratedSurfaces() ? TRUE : FALSE;
rv = NPERR_NO_ERROR;
break;
}
« no previous file with comments | « no previous file | webkit/glue/plugins/webplugin_delegate_impl_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698