[Win] Remove redundant macros that are set in the CMake config
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 May 2017 02:11:26 +0000 (02:11 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 5 May 2017 02:11:26 +0000 (02:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=171571

Patch by Don Olmstead <don.olmstead@am.sony.com> on 2017-05-04
Reviewed by Brent Fulgham.

.:

* Source/cmake/OptionsWin.cmake:

Source/JavaScriptCore:

* config.h:

Source/ThirdParty/ANGLE:

* CMakeLists.txt:

Source/WebCore:

No new tests. No change in behavior.

* WebCorePrefix.h:
* config.h:
* platform/win/COMPtr.h:
* platform/win/WindowsTouch.h:
* testing/js/WebCoreTestSupportPrefix.h:

Source/WebCore/PAL:

No new tests. No change in behavior.

* config.h:

Source/WebKit/win:

* COMEnumVariant.h:
* COMPropertyBag.h:
* WebKitPrefix.h:

Source/WTF:

* config.h:

Tools:

* DumpRenderTree/config.h:
* MiniBrowser/win/stdafx.h:
* TestWebKitAPI/config.h:

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

24 files changed:
ChangeLog
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/config.h
Source/ThirdParty/ANGLE/CMakeLists.txt
Source/ThirdParty/ANGLE/ChangeLog
Source/WTF/ChangeLog
Source/WTF/config.h
Source/WebCore/ChangeLog
Source/WebCore/PAL/ChangeLog
Source/WebCore/PAL/config.h
Source/WebCore/WebCorePrefix.h
Source/WebCore/config.h
Source/WebCore/platform/win/COMPtr.h
Source/WebCore/platform/win/WindowsTouch.h
Source/WebCore/testing/js/WebCoreTestSupportPrefix.h
Source/WebKit/win/COMEnumVariant.h
Source/WebKit/win/COMPropertyBag.h
Source/WebKit/win/ChangeLog
Source/WebKit/win/WebKitPrefix.h
Source/cmake/OptionsWin.cmake
Tools/ChangeLog
Tools/DumpRenderTree/config.h
Tools/MiniBrowser/win/stdafx.h
Tools/TestWebKitAPI/config.h

index b89dfb2..eaefb9c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * Source/cmake/OptionsWin.cmake:
+
 2017-05-03  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         Unreviewed. Update OptionsGTK.cmake and NEWS for 2.17.1 release.
index 655f419..c9fe6a2 100644 (file)
@@ -1,3 +1,12 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * config.h:
+
 2017-05-04  Mark Lam  <mark.lam@apple.com>
 
         Gardening: Build fix for Windows after r216217.
index bea1bf0..53c557e 100644 (file)
 #include "JSExportMacros.h"
 #endif
 
-#if OS(WINDOWS)
-
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
-#if !COMPILER(MSVC7_OR_LOWER)
-// We need to define this before the first #include of stdlib.h or it won't contain rand_s.
-#ifndef _CRT_RAND_S
-#define _CRT_RAND_S
-#endif
-#endif // !COMPILER(MSVC7_OR_LOWER
-
-#endif // OS(WINDOWS)
-
 #ifdef __cplusplus
 #undef new
 #undef delete
@@ -61,4 +42,3 @@
 #else
 #define SKIP_STATIC_CONSTRUCTORS_ON_GCC 1
 #endif
-
index 1a34b30..9b34f1b 100644 (file)
@@ -6,6 +6,13 @@ include_directories(
     include/KHR
 )
 
+if (WIN32)
+    # FIXME: DX11 support will not compile if this preprocessor definition is set
+    # DirectX Documentation is setting that version to 0x700 but there is no
+    # corresponding value in sdkddkver.h
+    remove_definitions(-D_WIN32_WINNT=0x601)
+endif ()
+
 set(ANGLEEGL_SOURCES
     src/libEGL/libEGL.cpp
 )
index 6a4c180..93b7be9 100644 (file)
@@ -1,3 +1,12 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * CMakeLists.txt:
+
 2017-04-20  Konstantin Tokarev  <annulen@yandex.ru>
 
         [cmake] Define FORWARDING_HEADERS_DIR in WebKitFS and use it everywhere
index a8b66c9..603aff2 100644 (file)
@@ -1,3 +1,12 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * config.h:
+
 2017-05-04  Mark Lam  <mark.lam@apple.com>
 
         NeverDestroyed<String>(ASCIILiteral(...)) is not thread safe.
index dbdef9b..36de392 100644 (file)
 #include <wtf/Platform.h>
 #include <wtf/ExportMacros.h>
 
-#if OS(WINDOWS)
-
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
-#if !COMPILER(MSVC7_OR_LOWER)
-// We need to define this before the first #include of stdlib.h or it won't contain rand_s.
-#ifndef _CRT_RAND_S
-#define _CRT_RAND_S
-#endif
-#endif
-
-#endif
-
 #ifdef __cplusplus
 #undef new
 #undef delete
index 9c274f4..2083c40 100644 (file)
@@ -1,3 +1,18 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        No new tests. No change in behavior.
+
+        * WebCorePrefix.h:
+        * config.h:
+        * platform/win/COMPtr.h:
+        * platform/win/WindowsTouch.h:
+        * testing/js/WebCoreTestSupportPrefix.h:
+
 2017-05-04  Tim Horton  <timothy_horton@apple.com>
 
         [Mac] Modernize image for dragged link
index d3d242d..ef97486 100644 (file)
@@ -1,3 +1,14 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        No new tests. No change in behavior.
+
+        * config.h:
+
 2017-05-04  Sam Weinig  <sam@webkit.org>
 
         Remove support for legacy Notifications
index 5488c80..a8c4731 100644 (file)
 
 #include <wtf/ExportMacros.h>
 
-#ifdef __APPLE__
-#define HAVE_FUNC_USLEEP 1
-#endif /* __APPLE__ */
-
-#if OS(WINDOWS)
-
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
-#endif /* OS(WINDOWS) */
-
 #ifdef __cplusplus
 
 // These undefs match up with defines in WebCorePrefix.h for Mac OS X.
index 6ee3020..4311c25 100644 (file)
 
 #if OS(WINDOWS)
 
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
 #if !USE(CURL)
 #ifndef _WINSOCKAPI_
 #define _WINSOCKAPI_ // Prevent inclusion of winsock.h in windows.h
 #undef try
 #undef catch
 #endif
-
index d5d43fe..e30f412 100644 (file)
 
 #if OS(WINDOWS)
 
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
 // CURL needs winsock, so don't prevent inclusion of it
 #if !USE(CURL)
 #ifndef _WINSOCKAPI_
index e595edf..7141fb4 100644 (file)
 #ifndef COMPtr_h
 #define COMPtr_h
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
-
 #include <unknwn.h>
 #include <wtf/Assertions.h>
 #include <wtf/HashTraits.h>
index bb482c8..658c665 100644 (file)
@@ -23,8 +23,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. 
  */
 
-#ifndef WindowsTouch_h
-#define WindowsTouch_h
+#pragma once
 
 /*
  * The following constants are used to determine multitouch and gesture behavior
 // Press and tap Configuration Flags
 #define GC_PRESSANDTAP 0x00000001
 #define GC_ROLLOVER GC_PRESSANDTAP
-
-#if WINVER < 0x0601
-
-// GESTUREINFO struct definition
-typedef struct tagGESTUREINFO {
-    UINT cbSize;                    // size, in bytes, of this structure (including variable length Args field)
-    DWORD dwFlags;                  // see GF_* flags
-    DWORD dwID;                     // gesture ID, see GID_* defines
-    HWND hwndTarget;                // handle to window targeted by this gesture
-    POINTS ptsLocation;             // current location of this gesture
-    DWORD dwInstanceID;             // internally used
-    DWORD dwSequenceID;             // internally used
-    ULONGLONG ullArguments;         // arguments for gestures whose arguments fit in 8 BYTES
-    UINT cbExtraArgs;               // size, in bytes, of extra arguments, if any, that accompany this gesture
-} GESTUREINFO, *PGESTUREINFO;
-typedef GESTUREINFO const * PCGESTUREINFO;
-
-// GESTURECONFIG struct defintion
-typedef struct tagGESTURECONFIG {
-    DWORD dwID;                     // gesture ID
-    DWORD dwWant;                   // settings related to gesture ID that are to be turned on
-    DWORD dwBlock;                  // settings related to gesture ID that are to be turned off
-} GESTURECONFIG, *PGESTURECONFIG;
-
-/*
- * Gesture notification structure
- *   - The WM_GESTURENOTIFY message lParam contains a pointer to this structure.
- *   - The WM_GESTURENOTIFY message notifies a window that gesture recognition is
- *     in progress and a gesture will be generated if one is recognized under the
- *     current gesture settings.
- */
-typedef struct tagGESTURENOTIFYSTRUCT {
-    UINT cbSize;                    // size, in bytes, of this structure
-    DWORD dwFlags;                  // unused
-    HWND hwndTarget;                // handle to window targeted by the gesture
-    POINTS ptsLocation;             // starting location
-    DWORD dwInstanceID;             // internally used
-} GESTURENOTIFYSTRUCT, *PGESTURENOTIFYSTRUCT;
-
-DECLARE_HANDLE(HGESTUREINFO);
-
-#endif
-
-#endif
index da9b0fc..7921e27 100644 (file)
 
 #if OS(WINDOWS)
 
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
 #if !USE(CURL)
 #ifndef _WINSOCKAPI_
 #define _WINSOCKAPI_ // Prevent inclusion of winsock.h in windows.h
index cba2ff6..2fe0f10 100644 (file)
 #ifndef COMEnumVariant_h
 #define COMEnumVariant_h
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
 #include <unknwn.h>
 
-
 #include "COMVariantSetter.h"
 
 template<typename ContainerType>
index ba8f534..7181f2b 100644 (file)
@@ -26,9 +26,6 @@
 #ifndef COMPropertyBag_h
 #define COMPropertyBag_h
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
 #include <ocidl.h>
 #include <unknwn.h>
 
index c3c9fcf..f0976bb 100644 (file)
@@ -1,3 +1,14 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * COMEnumVariant.h:
+        * COMPropertyBag.h:
+        * WebKitPrefix.h:
+
 2017-05-04  Sam Weinig  <sam@webkit.org>
 
         Remove support for legacy Notifications
index 802f1d1..bc369dc 100644 (file)
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */ 
 
-#ifndef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-#endif
-
-#ifndef WINVER
-#define WINVER 0x0601
-#endif
-
 #ifndef _WINSOCKAPI_
 #define _WINSOCKAPI_ // Prevent inclusion of winsock.h in windows.h
 #endif
@@ -55,4 +47,3 @@
 // as WEBCORE_EXPORT
 #undef WEBCORE_EXPORT
 #define WEBCORE_EXPORT WTF_EXPORT_DECLARATION
-
index dc2fc08..5d93ba0 100644 (file)
@@ -1,4 +1,11 @@
-add_definitions(-DNOMINMAX -DUNICODE -D_UNICODE -D_WINDOWS -DWINVER=0x601)
+# Define minimum supported Windows version
+# https://msdn.microsoft.com/en-us/library/6sehtctf.aspx
+#
+# Currently set to Windows 7
+add_definitions(-D_WINDOWS -DWINVER=0x601 -D_WIN32_WINNT=0x601)
+
+add_definitions(-DNOMINMAX)
+add_definitions(-DUNICODE -D_UNICODE)
 
 WEBKIT_OPTION_BEGIN()
 WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_3D_TRANSFORMS PUBLIC ON)
index 3caaefd..37b7dc6 100644 (file)
@@ -1,3 +1,14 @@
+2017-05-04  Don Olmstead  <don.olmstead@am.sony.com>
+
+        [Win] Remove redundant macros that are set in the CMake config
+        https://bugs.webkit.org/show_bug.cgi?id=171571
+
+        Reviewed by Brent Fulgham.
+
+        * DumpRenderTree/config.h:
+        * MiniBrowser/win/stdafx.h:
+        * TestWebKitAPI/config.h:
+
 2017-05-04  Said Abou-Hallawa  <sabouhallawa@apple.com>
 
         Add testRunner.display() to force displaying the web page for testing
index 3ee18a3..7450660 100644 (file)
 #define USE_CFURLCONNECTION 1
 #endif
 
-#undef _WIN32_WINNT
-#define _WIN32_WINNT 0x601
-
-#undef WINVER
-#define WINVER 0x0601
-
 #undef _WINSOCKAPI_
 #define _WINSOCKAPI_ // Prevent inclusion of winsock.h in windows.h
 #endif  // PLATFORM(WIN)
index 6eceb23..0e0c749 100644 (file)
 
 #pragma once
 
-// Modify the following defines if you have to target a platform prior to the ones specified below.
-// Refer to MSDN for the latest info on corresponding values for different platforms.
-#ifndef WINVER // Allow use of features specific to Windows 7 or later.
-#define WINVER 0x0601 // Change this to the appropriate value to target other versions of Windows.
-#endif
-
-#ifndef _WIN32_WINNT // Allow use of features specific to Windows 7 or later.
-#define _WIN32_WINNT 0x601 // Change this to the appropriate value to target other versions of Windows.
-#endif                        
-
-#ifndef _WIN32_WINDOWS // Allow use of features specific to Windows 7 or later.
-#define _WIN32_WINDOWS 0x601 // Change this to the appropriate value to target Windows 7 or later.
-#endif
-
-#ifndef _WIN32_IE            // Allow use of features specific to IE 6.0 or later.
-#define _WIN32_IE 0x0600    // Change this to the appropriate value to target other versions of IE.
-#endif
-
 #define WIN32_LEAN_AND_MEAN        // Exclude rarely-used stuff from Windows headers
 
 // Needed for limit defines, like INTMAX_MAX, which is used by the std C++ library
 #define __STDC_LIMIT_MACROS
 #endif
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
-
 #include <assert.h>
 #include <comip.h>
 #include <commctrl.h>
index c5ce18b..24ebcb6 100644 (file)
 
 #elif PLATFORM(WIN)
 
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
-
 #if PLATFORM(WIN_CAIRO)
 #undef USE_CG
 #define USE_CAIRO 1