Merge r175696 - [GTK] [Stable] webkitgtk 2.6.1 fails to load flashplugin

https://bugs.webkit.org/show_bug.cgi?id=137849

Reviewed by Carlos Garcia Campos.

NPPVpluginNeedsXEmbed is a boolean value, so it should normally
use an NPBool (1 byte). However some plugins (the Flash player in
particular) are using an int instead, so we have to do it as well
else we'll end up corrupting the stack.

* WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
(WebKit::NetscapePlugin::platformPostInitialize):

Source/WebKit2/ChangeLog

@@ -1,3 +1,18 @@
+2014-11-06  Alberto Garcia  <berto@igalia.com>
+
+        [GTK] [Stable] webkitgtk 2.6.1 fails to load flashplugin
+        https://bugs.webkit.org/show_bug.cgi?id=137849
+
+        Reviewed by Carlos Garcia Campos.
+
+        NPPVpluginNeedsXEmbed is a boolean value, so it should normally
+        use an NPBool (1 byte). However some plugins (the Flash player in
+        particular) are using an int instead, so we have to do it as well
+        else we'll end up corrupting the stack.
+
+        * WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp:
+        (WebKit::NetscapePlugin::platformPostInitialize):
+
 2014-11-18  Carlos Alberto Lopez Perez  <clopez@igalia.com>
 
         [SOUP] [GnuTLS] Don't use a SSL3.0 record version in client hello.

Source/WebKit2/WebProcess/Plugins/Netscape/x11/NetscapePluginX11.cpp

@@ -201,7 +201,9 @@ void NetscapePlugin::platformPreInitialize()
 bool NetscapePlugin::platformPostInitialize()
 {
     uint64_t windowID = 0;
-    bool needsXEmbed = false;
+    // NPPVpluginNeedsXEmbed is a boolean value, but at least the
+    // Flash player plugin is using an 'int' instead.
+    int needsXEmbed = 0;
     if (m_isWindowed) {
         NPP_GetValue(NPPVpluginNeedsXEmbed, &needsXEmbed);
         if (needsXEmbed) {