From b2e818917cad52c5b08b59475e56c1b8ca2c8e2f Mon Sep 17 00:00:00 2001 From: rdb Date: Sat, 13 Nov 2010 20:40:22 +0000 Subject: [PATCH] we should fix our XEmbed implementation before we think about support chromium --- direct/src/plugin_npapi/startup.cxx | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/direct/src/plugin_npapi/startup.cxx b/direct/src/plugin_npapi/startup.cxx index 7d8f0318d7..4bc1409d05 100644 --- a/direct/src/plugin_npapi/startup.cxx +++ b/direct/src/plugin_npapi/startup.cxx @@ -565,12 +565,15 @@ NPP_GetValue(NPP instance, NPPVariable variable, void *value) { } else if (variable == NPPVpluginNeedsXEmbed) { // We'll say yes if the browser supports it. // This is necessary to support Chromium. - NPBool supports_xembed = false; - NPError err = browser->getvalue(instance, NPNVSupportsXEmbedBool, &supports_xembed); - if (err != NPERR_NO_ERROR) { - supports_xembed = false; - } - *((NPBool *)value) = supports_xembed; + //NPBool supports_xembed = false; + //NPError err = browser->getvalue(instance, NPNVSupportsXEmbedBool, &supports_xembed); + //if (err != NPERR_NO_ERROR) { + // supports_xembed = false; + //} + // At the moment, setting it to true doesn't work + // at all on Linux. We'll have to fix that before + // we support Chromium, I suppose. + *((NPBool *)value) = false; return NPERR_NO_ERROR; } else { return NP_GetValue(NULL, variable, value);