Attachment 370016 Details for Bug 197934 – WIP Patch

[patch] WIP Patch

bug-197934.diff (text/plain), 17.07 KB, created by Don Olmstead on 2019-05-15 18:41:59 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Don Olmstead

Created: 2019-05-15 18:41:59 PDT

Size: 17.07 KB

patch

obsolete

>diff --git a/CMakeLists.txt b/CMakeLists.txt
>index d3e8a23f9ff..e79f41259e4 100644
>--- a/CMakeLists.txt
>+++ b/CMakeLists.txt
>@@ -2,13 +2,13 @@
> # Determine CMake version and build type.
> # -----------------------------------------------------------------------------
> #
>-# NOTE: cmake_minimum_required() and project() *MUST* be the two fist commands
>+# NOTE: cmake_minimum_required() and project() *MUST* be the two first commands
> # used, see https://cmake.org/cmake/help/v3.3/command/project.html -- the
> # latter in particular handles loading a bunch of shared CMake definitions
> # and loading the cross-compilation settings from CMAKE_TOOLCHAIN_FILE.
> #
> 
>-cmake_minimum_required(VERSION 3.3)
>+cmake_minimum_required(VERSION 3.7)
> 
> project(WebKit)
> 
>diff --git a/Source/JavaScriptCore/CMakeLists.txt b/Source/JavaScriptCore/CMakeLists.txt
>index 5d6d1701b34..e716c71682d 100644
>--- a/Source/JavaScriptCore/CMakeLists.txt
>+++ b/Source/JavaScriptCore/CMakeLists.txt
>@@ -54,10 +54,6 @@ if (USE_CAPSTONE)
>     list(APPEND JavaScriptCore_PRIVATE_INCLUDE_DIRECTORIES "${THIRDPARTY_DIR}/capstone/Source/include")
> endif ()
> 
>-set(JavaScriptCore_SYSTEM_INCLUDE_DIRECTORIES
>-    "${ICU_INCLUDE_DIRS}"
>-)
>-
> set(JavaScriptCore_OBJECT_LUT_SOURCES
>     runtime/ArrayConstructor.cpp
>     runtime/AsyncFromSyncIteratorPrototype.cpp
>@@ -121,7 +117,6 @@ set(JavaScriptCore_OBJECT_LUT_SOURCES
> 
> set(JavaScriptCore_LIBRARIES
>     WTF${DEBUG_SUFFIX}
>-    ${ICU_I18N_LIBRARIES}
> )
> 
> if (USE_CAPSTONE)
>diff --git a/Source/JavaScriptCore/PlatformWin.cmake b/Source/JavaScriptCore/PlatformWin.cmake
>index 425b97ecd34..c7d0ef5fbb4 100644
>--- a/Source/JavaScriptCore/PlatformWin.cmake
>+++ b/Source/JavaScriptCore/PlatformWin.cmake
>@@ -27,7 +27,6 @@ endif ()
> 
> if (NOT WTF_PLATFORM_WIN_CAIRO)
>     list(APPEND JavaScriptCore_LIBRARIES
>-        ${ICU_LIBRARIES}
>         winmm
>     )
> endif ()
>diff --git a/Source/WTF/wtf/CMakeLists.txt b/Source/WTF/wtf/CMakeLists.txt
>index 6d6408019cf..07f501ac65b 100644
>--- a/Source/WTF/wtf/CMakeLists.txt
>+++ b/Source/WTF/wtf/CMakeLists.txt
>@@ -491,13 +491,11 @@ list(APPEND WTF_SOURCES
> 
>     unicode/icu/CollatorICU.cpp
> )
>-set(WTF_SYSTEM_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>-)
>+
> list(APPEND WTF_LIBRARIES
>-    ${ICU_DATA_LIBRARIES}
>-    ${ICU_I18N_LIBRARIES}
>-    ${ICU_LIBRARIES}
>+    ICU::data
>+    ICU::i18n
>+    ICU::uc
> )
> 
> WEBKIT_FRAMEWORK_DECLARE(WTF)
>diff --git a/Source/WTF/wtf/PlatformPlayStation.cmake b/Source/WTF/wtf/PlatformPlayStation.cmake
>index 1a524b1d4d0..932b2a3da72 100644
>--- a/Source/WTF/wtf/PlatformPlayStation.cmake
>+++ b/Source/WTF/wtf/PlatformPlayStation.cmake
>@@ -17,7 +17,6 @@ list(APPEND WTF_SOURCES
> 
> list(APPEND WTF_LIBRARIES
>     ${CMAKE_THREAD_LIBS_INIT}
>-    ${ICU_LIBRARIES}
> 
>     ${C_STD_LIBRARY}
>     ${KERNEL_LIBRARY}
>diff --git a/Source/WebCore/CMakeLists.txt b/Source/WebCore/CMakeLists.txt
>index fd99e7dbd08..5ae59864376 100644
>--- a/Source/WebCore/CMakeLists.txt
>+++ b/Source/WebCore/CMakeLists.txt
>@@ -166,7 +166,6 @@ set(WebCore_PRIVATE_INCLUDE_DIRECTORIES
> )
> 
> set(WebCore_SYSTEM_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>     ${LIBXML2_INCLUDE_DIR}
>     ${LIBXSLT_INCLUDE_DIR}
>     ${SQLITE_INCLUDE_DIR}
>@@ -1196,11 +1195,11 @@ set(WebCore_PLUG_INS_RESOURCES
> )
> 
> set(WebCore_LIBRARIES
>-    ${ICU_LIBRARIES}
>     ${LIBXML2_LIBRARIES}
>     ${LIBXSLT_LIBRARIES}
>     ${SQLITE_LIBRARIES}
>     ${ZLIB_LIBRARIES}
>+    JavaScriptCore
>     PAL${DEBUG_SUFFIX}
> )
> 
>diff --git a/Source/WebCore/PAL/pal/CMakeLists.txt b/Source/WebCore/PAL/pal/CMakeLists.txt
>index c50c485b3a7..a843006a9c0 100644
>--- a/Source/WebCore/PAL/pal/CMakeLists.txt
>+++ b/Source/WebCore/PAL/pal/CMakeLists.txt
>@@ -29,6 +29,7 @@ set(PAL_SOURCES
> 
> set(PAL_PRIVATE_INCLUDE_DIRECTORIES
>     "${CMAKE_BINARY_DIR}"
>+    "${WTF_FRAMEWORK_HEADERS_DIR}"
>     "${PAL_DERIVED_SOURCES_DIR}"
>     "${PAL_DIR}"
>     "${PAL_DIR}/pal"
>@@ -37,12 +38,8 @@ set(PAL_PRIVATE_INCLUDE_DIRECTORIES
>     "${PAL_DIR}/pal/text"
> )
> 
>-set(PAL_SYSTEM_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>-)
>-
> set(PAL_LIBRARIES
>-    JavaScriptCore${DEBUG_SUFFIX}
>+    WTF
> )
> 
> WEBKIT_FRAMEWORK_DECLARE(PAL)
>diff --git a/Source/WebCore/PAL/pal/SessionID.h b/Source/WebCore/PAL/pal/SessionID.h
>index 2b1cdb29874..96c99d02486 100644
>--- a/Source/WebCore/PAL/pal/SessionID.h
>+++ b/Source/WebCore/PAL/pal/SessionID.h
>@@ -66,7 +66,7 @@ public:
>     template<class Decoder> static bool decode(Decoder&, SessionID&);
>     template<class Decoder> static Optional<SessionID> decode(Decoder&);
> 
>-    SessionID isolatedCopy() const;
>+    PAL_EXPORT SessionID isolatedCopy() const;
> 
> private:
>     explicit SessionID(uint64_t sessionID)
>diff --git a/Source/WebCore/PAL/pal/system/Clock.h b/Source/WebCore/PAL/pal/system/Clock.h
>index 50860adc793..42c9bab3047 100644
>--- a/Source/WebCore/PAL/pal/system/Clock.h
>+++ b/Source/WebCore/PAL/pal/system/Clock.h
>@@ -33,7 +33,7 @@ namespace PAL {
> class Clock {
>     WTF_MAKE_FAST_ALLOCATED;
> public:
>-    static std::unique_ptr<Clock> create();
>+    PAL_EXPORT static std::unique_ptr<Clock> create();
>     virtual ~Clock() = default;
> 
>     virtual void setCurrentTime(double) = 0;
>diff --git a/Source/WebCore/PAL/pal/system/SystemSleepListener.h b/Source/WebCore/PAL/pal/system/SystemSleepListener.h
>index b96cab199c7..90f4b3c2a03 100644
>--- a/Source/WebCore/PAL/pal/system/SystemSleepListener.h
>+++ b/Source/WebCore/PAL/pal/system/SystemSleepListener.h
>@@ -36,7 +36,7 @@ public:
>         virtual void systemDidWake() = 0;
>     };
> 
>-    static std::unique_ptr<SystemSleepListener> create(Client&);
>+    PAL_EXPORT static std::unique_ptr<SystemSleepListener> create(Client&);
>     virtual ~SystemSleepListener() = default;
> 
>     Client& client() { return m_client; }
>diff --git a/Source/WebCore/PlatformPlayStation.cmake b/Source/WebCore/PlatformPlayStation.cmake
>index d1ea46ce800..7193bc5ae10 100644
>--- a/Source/WebCore/PlatformPlayStation.cmake
>+++ b/Source/WebCore/PlatformPlayStation.cmake
>@@ -81,7 +81,6 @@ set(WebCore_USER_AGENT_SCRIPTS
> )
> 
> list(APPEND WebCore_SYSTEM_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>     ${LIBXML2_INCLUDE_DIR}
>     ${SQLITE_INCLUDE_DIR}
>     ${ZLIB_INCLUDE_DIRS}
>@@ -89,7 +88,6 @@ list(APPEND WebCore_SYSTEM_INCLUDE_DIRECTORIES
> )
> 
> list(APPEND WebCore_LIBRARIES
>-    ${ICU_LIBRARIES}
>     ${LIBXML2_LIBRARIES}
>     ${SQLITE_LIBRARIES}
>     ${ZLIB_LIBRARIES}
>diff --git a/Source/WebCore/PlatformWPE.cmake b/Source/WebCore/PlatformWPE.cmake
>index 1cad0e617fb..4100c815259 100644
>--- a/Source/WebCore/PlatformWPE.cmake
>+++ b/Source/WebCore/PlatformWPE.cmake
>@@ -52,7 +52,6 @@ list(APPEND WebCore_LIBRARIES
>     ${GLIB_GMODULE_LIBRARIES}
>     ${GLIB_GOBJECT_LIBRARIES}
>     ${GLIB_LIBRARIES}
>-    ${ICU_LIBRARIES}
>     ${LIBTASN1_LIBRARIES}
>     ${UPOWERGLIB_LIBRARIES}
>     ${WPE_LIBRARIES}
>@@ -61,7 +60,6 @@ list(APPEND WebCore_LIBRARIES
> list(APPEND WebCore_SYSTEM_INCLUDE_DIRECTORIES
>     ${GIO_UNIX_INCLUDE_DIRS}
>     ${GLIB_INCLUDE_DIRS}
>-    ${ICU_INCLUDE_DIRS}
>     ${LIBTASN1_INCLUDE_DIRS}
>     ${UPOWERGLIB_INCLUDE_DIRS}
>     ${WPE_INCLUDE_DIRS}
>diff --git a/Source/WebDriver/PlatformWPE.cmake b/Source/WebDriver/PlatformWPE.cmake
>index ba2179898a5..bd3cd811bd4 100644
>--- a/Source/WebDriver/PlatformWPE.cmake
>+++ b/Source/WebDriver/PlatformWPE.cmake
>@@ -2,7 +2,6 @@ set(WebDriver_Process_OUTPUT_NAME WPEWebDriver)
> 
> list(APPEND WebDriver_SYSTEM_INCLUDE_DIRECTORIES
>     "${GLIB_INCLUDE_DIRS}"
>-    "${ICU_INCLUDE_DIRS}"
>     "${LIBSOUP_INCLUDE_DIRS}"
> )
> 
>diff --git a/Source/WebKit/CMakeLists.txt b/Source/WebKit/CMakeLists.txt
>index 4b0bec7cb80..310f1698cf1 100644
>--- a/Source/WebKit/CMakeLists.txt
>+++ b/Source/WebKit/CMakeLists.txt
>@@ -98,10 +98,6 @@ set(WebKit_INCLUDE_DIRECTORIES
> 
> set(PROTOCOL_GENERATOR_SCRIPTS_DIR "${JAVASCRIPTCORE_DIR}/inspector/scripts")
> 
>-set(WebKit_SYSTEM_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>-)
>-
> list(APPEND WebKit_UNIFIED_SOURCE_LIST_FILES
>     "Sources.txt"
> )
>diff --git a/Source/WebKitLegacy/CMakeLists.txt b/Source/WebKitLegacy/CMakeLists.txt
>index b0c77ddebff..960a970ebb0 100644
>--- a/Source/WebKitLegacy/CMakeLists.txt
>+++ b/Source/WebKitLegacy/CMakeLists.txt
>@@ -25,10 +25,6 @@ set(WebKitLegacy_PRIVATE_INCLUDE_DIRECTORIES
>     "${WebKitLegacy_DERIVED_SOURCES_DIR}"
> )
> 
>-set(WebKitLegacy_SYSTEM_INCLUDE_DIRECTORIES
>-    "${ICU_INCLUDE_DIRS}"
>-)
>-
> set(WebKitLegacy_LIBRARIES
>     PRIVATE JavaScriptCore${DEBUG_SUFFIX}
>     PRIVATE WebCore${DEBUG_SUFFIX}
>diff --git a/Source/cmake/FindICU.cmake b/Source/cmake/FindICU.cmake
>deleted file mode 100644
>index f9d79c19143..00000000000
>--- a/Source/cmake/FindICU.cmake
>+++ /dev/null
>@@ -1,110 +0,0 @@
>-# Finds the International Components for Unicode (ICU) Library
>-#
>-#  ICU_FOUND          - True if ICU found.
>-#  ICU_I18N_FOUND     - True if ICU's internationalization library found.
>-#  ICU_INCLUDE_DIRS   - Directory to include to get ICU headers
>-#                       Note: always include ICU headers as, e.g.,
>-#                       unicode/utypes.h
>-#  ICU_LIBRARIES      - Libraries to link against for the common ICU
>-#  ICU_I18N_LIBRARIES - Libraries to link against for ICU internationaliation
>-#                       (note: in addition to ICU_LIBRARIES)
>-
>-find_package(PkgConfig)
>-pkg_check_modules(PC_ICU icu-uc)
>-
>-# Look for the header file.
>-find_path(
>-    ICU_INCLUDE_DIR
>-    NAMES unicode/utypes.h
>-    HINTS ${PC_ICU_INCLUDE_DIRS}
>-          ${PC_ICU_INCLUDEDIR}
>-          ${WEBKIT_LIBRARIES_INCLUDE_DIR}
>-    DOC "Include directory for the ICU library")
>-mark_as_advanced(ICU_INCLUDE_DIR)
>-
>-# Look for the library.
>-find_library(
>-    ICU_LIBRARY
>-    NAMES libicuuc cygicuuc cygicuuc32 icuuc
>-    HINTS ${PC_ICU_LIBRARY_DIRS}
>-          ${PC_ICU_LIBDIR}
>-          ${WEBKIT_LIBRARIES_LINK_DIR}
>-    DOC "Libraries to link against for the common parts of ICU")
>-mark_as_advanced(ICU_LIBRARY)
>-
>-# Copy the results to the output variables.
>-if (ICU_INCLUDE_DIR AND ICU_LIBRARY)
>-    set(ICU_FOUND 1)
>-    set(ICU_LIBRARIES ${ICU_LIBRARY})
>-    set(ICU_INCLUDE_DIRS ${ICU_INCLUDE_DIR})
>-
>-    set(ICU_VERSION 0)
>-    set(ICU_MAJOR_VERSION 0)
>-    set(ICU_MINOR_VERSION 0)
>-    file(READ "${ICU_INCLUDE_DIR}/unicode/uversion.h" _ICU_VERSION_CONENTS)
>-    string(REGEX REPLACE ".*#define U_ICU_VERSION_MAJOR_NUM ([0-9]+).*" "\\1" ICU_MAJOR_VERSION "${_ICU_VERSION_CONENTS}")
>-    string(REGEX REPLACE ".*#define U_ICU_VERSION_MINOR_NUM ([0-9]+).*" "\\1" ICU_MINOR_VERSION "${_ICU_VERSION_CONENTS}")
>-
>-    set(ICU_VERSION "${ICU_MAJOR_VERSION}.${ICU_MINOR_VERSION}")
>-
>-    # Look for the ICU internationalization libraries
>-    pkg_check_modules(PC_ICU_I18N icu-i18n)
>-    find_library(
>-        ICU_I18N_LIBRARY
>-        NAMES icui18n libicui18n libicuin cygicuin cygicuin32 icuin
>-        HINTS ${PC_ICU_I18N_LIBRARY_DIRS}
>-              ${PC_ICU_I18N_LIBDIR}
>-              ${WEBKIT_LIBRARIES_LINK_DIR}
>-        DOC "Libraries to link against for ICU internationalization")
>-    mark_as_advanced(ICU_I18N_LIBRARY)
>-    if (ICU_I18N_LIBRARY)
>-        set(ICU_I18N_FOUND 1)
>-        set(ICU_I18N_LIBRARIES ${ICU_I18N_LIBRARY})
>-    else ()
>-        set(ICU_I18N_FOUND 0)
>-        set(ICU_I18N_LIBRARIES)
>-    endif ()
>-
>-    # Look for the ICU data libraries
>-    find_library(
>-        ICU_DATA_LIBRARY
>-        NAMES icudata libicudata cygicudata cygicudata32
>-        HINTS ${PC_ICU_I18N_LIBRARY_DIRS}
>-              ${PC_ICU_I18N_LIBDIR}
>-              ${WEBKIT_LIBRARIES_LINK_DIR}
>-        DOC "Libraries to link against for ICU data")
>-    mark_as_advanced(ICU_DATA_LIBRARY)
>-    if (ICU_DATA_LIBRARY)
>-        set(ICU_DATA_FOUND 1)
>-        set(ICU_DATA_LIBRARIES ${ICU_DATA_LIBRARY})
>-    else ()
>-        set(ICU_DATA_FOUND 0)
>-        set(ICU_DATA_LIBRARIES)
>-    endif ()
>-else ()
>-    set(ICU_FOUND 0)
>-    set(ICU_I18N_FOUND 0)
>-    set(ICU_DATA_FOUND 0)
>-    set(ICU_LIBRARIES)
>-    set(ICU_I18N_LIBRARIES)
>-    set(ICU_DATA_LIBRARIES)
>-    set(ICU_INCLUDE_DIRS)
>-    set(ICU_VERSION)
>-    set(ICU_MAJOR_VERSION)
>-    set(ICU_MINOR_VERSION)
>-endif ()
>-
>-if (ICU_FOUND)
>-    if (NOT ICU_FIND_QUIETLY)
>-        message(STATUS "Found ICU header files in ${ICU_INCLUDE_DIRS}")
>-        message(STATUS "Found ICU libraries: ${ICU_LIBRARIES}")
>-        message(STATUS "Found ICU internationaliation libraries: ${ICU_I18N_LIBRARIES}")
>-        message(STATUS "Found ICU data libraries: ${ICU_DATA_LIBRARIES}")
>-    endif ()
>-else ()
>-    if (ICU_FIND_REQUIRED)
>-        message(FATAL_ERROR "Could not find ICU")
>-    else ()
>-        message(STATUS "Optional package ICU was not found")
>-    endif ()
>-endif ()
>diff --git a/Source/cmake/OptionsGTK.cmake b/Source/cmake/OptionsGTK.cmake
>index d290beffdd9..b21a58f52bc 100644
>--- a/Source/cmake/OptionsGTK.cmake
>+++ b/Source/cmake/OptionsGTK.cmake
>@@ -24,7 +24,7 @@ find_package(LibGcrypt 1.6.0 REQUIRED)
> find_package(GTK3 3.6.0 REQUIRED)
> find_package(GDK3 3.6.0 REQUIRED)
> find_package(HarfBuzz 0.9.2 REQUIRED)
>-find_package(ICU REQUIRED)
>+find_package(ICU REQUIRED COMPONENTS data i18n uc)
> find_package(JPEG REQUIRED)
> find_package(LibSoup 2.42.0 REQUIRED)
> find_package(LibXml2 2.8.0 REQUIRED)
>diff --git a/Source/cmake/OptionsJSCOnly.cmake b/Source/cmake/OptionsJSCOnly.cmake
>index 2affab7250d..ae3c2ea5f5e 100644
>--- a/Source/cmake/OptionsJSCOnly.cmake
>+++ b/Source/cmake/OptionsJSCOnly.cmake
>@@ -96,7 +96,7 @@ else ()
> endif ()
> 
> if (NOT APPLE)
>-    find_package(ICU REQUIRED)
>+    find_package(ICU REQUIRED COMPONENTS data i18n uc)
> else ()
>     add_definitions(-DU_DISABLE_RENAMING=1 -DU_SHOW_CPLUSPLUS_API=0)
>     set(ICU_INCLUDE_DIRS ${CMAKE_BINARY_DIR}/ICU/Headers)
>diff --git a/Source/cmake/OptionsPlayStation.cmake b/Source/cmake/OptionsPlayStation.cmake
>index a4e879339ac..5af7dc6ac1d 100644
>--- a/Source/cmake/OptionsPlayStation.cmake
>+++ b/Source/cmake/OptionsPlayStation.cmake
>@@ -80,7 +80,7 @@ find_package(EGL REQUIRED)
> find_package(Fontconfig REQUIRED)
> find_package(Freetype REQUIRED)
> find_package(HarfBuzz REQUIRED)
>-find_package(ICU REQUIRED)
>+find_package(ICU REQUIRED COMPONENTS data i18n uc)
> find_package(JPEG REQUIRED)
> find_package(LibPSL REQUIRED)
> find_package(LibXml2 REQUIRED)
>@@ -125,9 +125,10 @@ SET_AND_EXPOSE_TO_BUILD(USE_TEXTURE_MAPPER ON)
> SET_AND_EXPOSE_TO_BUILD(USE_TEXTURE_MAPPER_GL ON)
> SET_AND_EXPOSE_TO_BUILD(USE_TILED_BACKING_STORE ON)
> 
>-set(WTF_LIBRARY_TYPE STATIC)
>-set(JavaScriptCore_LIBRARY_TYPE STATIC)
>-set(WebCore_LIBRARY_TYPE STATIC)
>+set(WTF_LIBRARY_TYPE SHARED)
>+set(JavaScriptCore_LIBRARY_TYPE SHARED)
>+set(PAL_LIBRARY_TYPE SHARED)
>+set(WebCore_LIBRARY_TYPE SHARED)
> 
> set(ENABLE_API_TESTS ON)
> set(ENABLE_WEBCORE ON)
>diff --git a/Source/cmake/OptionsWPE.cmake b/Source/cmake/OptionsWPE.cmake
>index 5f7f2f62dce..c71456ae7bf 100644
>--- a/Source/cmake/OptionsWPE.cmake
>+++ b/Source/cmake/OptionsWPE.cmake
>@@ -17,7 +17,7 @@ find_package(Fontconfig 2.8.0 REQUIRED)
> find_package(Freetype 2.4.2 REQUIRED)
> find_package(GLIB 2.40.0 REQUIRED COMPONENTS gio gio-unix gobject gthread gmodule)
> find_package(HarfBuzz 0.9.18 REQUIRED)
>-find_package(ICU REQUIRED)
>+find_package(ICU REQUIRED COMPONENTS data i18n uc)
> find_package(JPEG REQUIRED)
> find_package(LibEpoxy 1.4.0 REQUIRED)
> find_package(LibGcrypt 1.6.0 REQUIRED)
>diff --git a/Source/cmake/OptionsWin.cmake b/Source/cmake/OptionsWin.cmake
>index b6c13862321..3e8f76977a5 100644
>--- a/Source/cmake/OptionsWin.cmake
>+++ b/Source/cmake/OptionsWin.cmake
>@@ -164,8 +164,6 @@ if (NOT ENABLE_UNIFIED_BUILDS AND WebCore_LIBRARY_TYPE MATCHES STATIC)
>     set(WebCore_LIBRARY_TYPE OBJECT)
> endif ()
> 
>-find_package(ICU REQUIRED)
>-
> # If <winsock2.h> is not included before <windows.h> redefinition errors occur
> # unless _WINSOCKAPI_ is defined before <windows.h> is included
> add_definitions(-D_WINSOCKAPI_=)
>diff --git a/Source/cmake/OptionsWinCairo.cmake b/Source/cmake/OptionsWinCairo.cmake
>index a716ec6f0ea..0fbad9545c0 100644
>--- a/Source/cmake/OptionsWinCairo.cmake
>+++ b/Source/cmake/OptionsWinCairo.cmake
>@@ -4,6 +4,7 @@ include(OptionsWin)
> 
> find_package(Cairo 1.15.12 REQUIRED)
> find_package(CURL 7.60.0 REQUIRED)
>+find_package(ICU REQUIRED COMPONENTS data i18n uc)
> find_package(JPEG 1.5.2 REQUIRED)
> find_package(LibXml2 2.9.7 REQUIRED)
> find_package(OpenSSL 2.0.0 REQUIRED)
>diff --git a/Tools/TestWebKitAPI/PlatformMac.cmake b/Tools/TestWebKitAPI/PlatformMac.cmake
>index 4e295d64c7d..38925dc89b4 100644
>--- a/Tools/TestWebKitAPI/PlatformMac.cmake
>+++ b/Tools/TestWebKitAPI/PlatformMac.cmake
>@@ -17,7 +17,6 @@ list(APPEND test_wtf_LIBRARIES
>     ${CARBON_LIBRARY}
>     ${COCOA_LIBRARY}
>     ${COREFOUNDATION_LIBRARY}
>-    ${ICU_LIBRARIES}
> )
> 
> list(APPEND TestWebKitAPI_LIBRARIES
>diff --git a/Tools/TestWebKitAPI/PlatformPlayStation.cmake b/Tools/TestWebKitAPI/PlatformPlayStation.cmake
>index abb8690f80a..952d043f062 100644
>--- a/Tools/TestWebKitAPI/PlatformPlayStation.cmake
>+++ b/Tools/TestWebKitAPI/PlatformPlayStation.cmake
>@@ -10,10 +10,6 @@ list(APPEND TestWTF_SOURCES
>     generic/UtilitiesGeneric.cpp
> )
> 
>-list(APPEND TestWTF_PRIVATE_INCLUDE_DIRECTORIES
>-    ${ICU_INCLUDE_DIRS}
>-)
>-
> list(APPEND TestWebCore_SOURCES
>     ${test_main_SOURCES}
> )
>diff --git a/Tools/WebKitTestRunner/CMakeLists.txt b/Tools/WebKitTestRunner/CMakeLists.txt
>index 34ff64aaf8e..632aa0b0675 100644
>--- a/Tools/WebKitTestRunner/CMakeLists.txt
>+++ b/Tools/WebKitTestRunner/CMakeLists.txt
>@@ -51,7 +51,6 @@ set(WebKitTestRunner_INCLUDE_DIRECTORIES
> 
> set(WebKitTestRunner_SYSTEM_INCLUDE_DIRECTORIES
>     ${LIBSOUP_INCLUDE_DIRS}
>-    ${ICU_INCLUDE_DIRS}
> )
> 
> set(WebKitTestRunnerInjectedBundle_SOURCES

Actions: View | Formatted Diff | Diff

Attachments on bug 197934: 370016 | 370152