.: [Gtk] Added MOZ_X11 build flag for TARGET_X11
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Apr 2012 05:30:39 +0000 (05:30 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Apr 2012 05:30:39 +0000 (05:30 +0000)
[Qt]  Added MOZ_X11 build flag for !embedded
[CMake]  Added MOZ_X11 build flag for WTF_OS_UNIX
https://bugs.webkit.org/show_bug.cgi?id=40785

Patch by Sriram Neelakandan <sriram.neelakandan@gmail.com> on 2012-04-22
Reviewed by Anders Carlsson.

* GNUmakefile.am:
* Source/cmake/OptionsCommon.cmake:

Source/WebCore: Sync the MOZ_X11 changes from r14 of http://code.google.com/p/npapi-sdk/
also, modified required build flags for Qt,Gtk and CMake
[Qt]  Added MOZ_X11 build flag for !embedded
https://bugs.webkit.org/show_bug.cgi?id=40785

Patch by Sriram Neelakandan <sriram.neelakandan@gmail.com> on 2012-04-22
Reviewed by Anders Carlsson.

No new tests. This does not change functionality. Affects all X11 Ports. Build should break if MOZ_X11 is not passed where required

* WebCore.pri:
* plugins/npapi.h:

Source/WebKit2: [Gtk] Added MOZ_X11 build flag for TARGET_X11
[Qt]  Added MOZ_X11 build flag for !embedded
https://bugs.webkit.org/show_bug.cgi?id=40785

Patch by Sriram Neelakandan <sriram.neelakandan@gmail.com> on 2012-04-22
Reviewed by Anders Carlsson.

* GNUmakefile.am:
* Target.pri:

Tools: [Gtk] Added MOZ_X11 build flag for TARGET_X11
https://bugs.webkit.org/show_bug.cgi?id=40785

Patch by Sriram Neelakandan <sriram.neelakandan@gmail.com> on 2012-04-22
Reviewed by Anders Carlsson.

* GNUmakefile.am:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@114869 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
GNUmakefile.am
Source/WebCore/ChangeLog
Source/WebCore/WebCore.pri
Source/WebCore/plugins/npapi.h
Source/WebKit2/ChangeLog
Source/WebKit2/GNUmakefile.am
Source/WebKit2/Target.pri
Source/cmake/OptionsCommon.cmake
Tools/ChangeLog
Tools/GNUmakefile.am

index b476ce2..4ea2e06 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,15 @@
+2012-04-22  Sriram Neelakandan  <sriram.neelakandan@gmail.com>
+
+        [Gtk] Added MOZ_X11 build flag for TARGET_X11
+        [Qt]  Added MOZ_X11 build flag for !embedded
+        [CMake]  Added MOZ_X11 build flag for WTF_OS_UNIX
+        https://bugs.webkit.org/show_bug.cgi?id=40785
+
+        Reviewed by Anders Carlsson.
+
+        * GNUmakefile.am:
+        * Source/cmake/OptionsCommon.cmake:
+
 2012-04-22  Adrian Bunk  <bunk@stusta.de>
 
         [GTK] Remove the obsolete Hildon UI extensions
index 65e7c85..75ca1e2 100644 (file)
@@ -154,6 +154,12 @@ global_cppflags += \
        -DXP_UNIX
 endif
 
+# Add MOZ_X11 for X11 targets only
+if TARGET_X11
+global_cppflags += \
+       -DMOZ_X11
+endif
+
 # Use the wide character version of win32 API by default
 if TARGET_WIN32
 global_cppflags += \
index 64635e3..08dc621 100644 (file)
@@ -1,3 +1,17 @@
+2012-04-22  Sriram Neelakandan  <sriram.neelakandan@gmail.com>
+
+        Sync the MOZ_X11 changes from r14 of http://code.google.com/p/npapi-sdk/
+        also, modified required build flags for Qt,Gtk and CMake
+        [Qt]  Added MOZ_X11 build flag for !embedded
+        https://bugs.webkit.org/show_bug.cgi?id=40785
+
+        Reviewed by Anders Carlsson.
+
+        No new tests. This does not change functionality. Affects all X11 Ports. Build should break if MOZ_X11 is not passed where required
+
+        * WebCore.pri:
+        * plugins/npapi.h:
+
 2012-04-22  Pablo Flouret  <pablof@motorola.com>
 
         Reset event propagation and canceled flags in Event.initEvent
index eba26dd..b430fcb 100644 (file)
@@ -137,6 +137,7 @@ contains(DEFINES, ENABLE_NETSCAPE_PLUGIN_API=1) {
             !embedded {
                 CONFIG += x11
                 LIBS += -lXrender
+                DEFINES += MOZ_X11
             }
             DEFINES += XP_UNIX
             DEFINES += ENABLE_NETSCAPE_PLUGIN_METADATA_CACHE=1
index 1de47e8..305a162 100644 (file)
 #endif
 
 #if defined(XP_UNIX)
+#include <stdio.h>
+#if defined(MOZ_X11)
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
-#include <stdio.h>
+#endif
 #endif
 
 /*----------------------------------------------------------------------*/
@@ -266,10 +268,12 @@ typedef struct
 typedef struct
 {
   int32_t      type;
+#if defined(MOZ_X11)
   Display*     display;
   Visual*      visual;
   Colormap     colormap;
   unsigned int depth;
+#endif
 } NPSetWindowCallbackStruct;
 
 typedef struct
@@ -535,7 +539,7 @@ typedef struct _NPEvent
   uint32_t wParam;
   uint32_t lParam;
 } NPEvent;
-#elif defined(XP_UNIX)
+#elif defined(XP_UNIX) && defined(MOZ_X11)
 typedef XEvent NPEvent;
 #else
 typedef void*  NPEvent;
@@ -549,7 +553,7 @@ typedef RgnHandle NPQDRegion;
 typedef CGPathRef NPCGRegion;
 #elif defined(XP_WIN)
 typedef HRGN NPRegion;
-#elif defined(XP_UNIX)
+#elif defined(XP_UNIX) && defined(MOZ_X11)
 typedef Region NPRegion;
 #else
 typedef void *NPRegion;
index f348fcb..15ac83a 100644 (file)
@@ -1,3 +1,14 @@
+2012-04-22  Sriram Neelakandan  <sriram.neelakandan@gmail.com>
+
+        [Gtk] Added MOZ_X11 build flag for TARGET_X11
+        [Qt]  Added MOZ_X11 build flag for !embedded
+        https://bugs.webkit.org/show_bug.cgi?id=40785
+
+        Reviewed by Anders Carlsson.
+
+        * GNUmakefile.am:
+        * Target.pri:
+
 2012-04-22  Jon Lee  <jonlee@apple.com>
 
         Remove notifications support on Mac Lion.
index 1df1310..2dafcaf 100644 (file)
@@ -1111,6 +1111,12 @@ libwebkit2gtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_CPP
        -DXP_UNIX
 endif
 
+# Add MOZ_X11 for X11 targets only
+if TARGET_X11
+libwebkit2gtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_CPPFLAGS += \
+       -DMOZ_X11
+endif
+
 # Plugin process
 if ENABLE_PLUGIN_PROCESS
 libwebkit2gtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_CPPFLAGS += \
index d8609d0..fd52029 100644 (file)
@@ -791,6 +791,7 @@ contains(DEFINES, ENABLE_GEOLOCATION=1): QT += location
 plugin_backend_xlib {
     DEFINES += XP_UNIX
     PKGCONFIG += x11
+    DEFINES += MOZ_X11
 }
 
 include(DerivedSources.pri)
index 37cb4a8..bae6527 100644 (file)
@@ -3,6 +3,8 @@ ADD_DEFINITIONS(-DHAVE_CONFIG_H=1)
 
 IF (WTF_OS_UNIX)
     ADD_DEFINITIONS(-DXP_UNIX)
+    #TODO: check for X11 target
+    ADD_DEFINITIONS(-DMOZ_X11) 
 ENDIF (WTF_OS_UNIX)
 
 # CODE_GENERATOR_PREPROCESSOR_WITH_LINEMARKERS only matters with GCC >= 4.7.0.  Since this
index 0187e63..1711395 100644 (file)
@@ -1,3 +1,12 @@
+2012-04-22  Sriram Neelakandan  <sriram.neelakandan@gmail.com>
+
+        [Gtk] Added MOZ_X11 build flag for TARGET_X11
+        https://bugs.webkit.org/show_bug.cgi?id=40785
+
+        Reviewed by Anders Carlsson.
+
+        * GNUmakefile.am:
+
 2012-04-22  Zan Dobersek  <zandobersek@gmail.com>
 
         [TestResultsServer] Impossible to inspect builds on non-Chromium builders
index 8e041f6..a101bec 100644 (file)
@@ -201,6 +201,11 @@ if !TARGET_WIN32
 TestNetscapePlugin_libtestnetscapeplugin_la_CPPFLAGS += -DXP_UNIX
 endif
 
+# Add MOZ_X11 only for X11 targets
+if TARGET_X11
+TestNetscapePlugin_libtestnetscapeplugin_la_CPPFLAGS += -DMOZ_X11
+endif
+
 TestNetscapePlugin_libtestnetscapeplugin_la_SOURCES = \
        Tools/DumpRenderTree/TestNetscapePlugIn/Tests/DocumentOpenInDestroyStream.cpp \
        Tools/DumpRenderTree/TestNetscapePlugIn/Tests/EvaluateJSAfterRemovingPluginElement.cpp \