Don't expose new semantic -apple-system color keywords on iOS.
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Jun 2018 04:25:05 +0000 (04:25 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Jun 2018 04:25:05 +0000 (04:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187080
rdar://problem/41505699

Reviewed by Tim Horton.

* DerivedSources.make: Use gnu++14, since gnu++17 is giving errors on macOS 10.12.
Source/WebCore:

* css/CSSValueKeywords.in: Define new semantic colors only on macOS.

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

Source/WebCore/ChangeLog
Source/WebCore/DerivedSources.make
Source/WebCore/css/CSSValueKeywords.in
Source/WebKit/ChangeLog
Source/WebKit/DerivedSources.make

index 0c9239c..4a6bfa5 100644 (file)
@@ -1,3 +1,14 @@
+2018-06-27  Timothy Hatcher  <timothy@apple.com>
+
+        Don't expose new semantic -apple-system color keywords on iOS.
+        https://bugs.webkit.org/show_bug.cgi?id=187080
+        rdar://problem/41505699
+
+        Reviewed by Tim Horton.
+
+        * DerivedSources.make: Use gnu++14, since gnu++17 is giving errors on macOS 10.12.
+        * css/CSSValueKeywords.in: Define new semantic colors only on macOS.
+
 2018-06-27  Zalan Bujtas  <zalan@apple.com>
 
         [LFC] Compute both the collapsed and the non-collapsed margin values.
index cddfdcd..239f1ae 100644 (file)
@@ -1017,25 +1017,25 @@ ifeq ($(USE_LLVM_TARGET_TRIPLES_FOR_CLANG),YES)
        TARGET_TRIPLE_FLAGS=-target $(CURRENT_ARCH)-$(LLVM_TARGET_TRIPLE_VENDOR)-$(LLVM_TARGET_TRIPLE_OS_VERSION)$(LLVM_TARGET_TRIPLE_SUFFIX)
 endif
 
-ifeq ($(shell $(CC) -std=gnu++17 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_IOS ' | cut -d' ' -f3), 1)
+ifeq ($(shell $(CC) -std=gnu++14 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_IOS ' | cut -d' ' -f3), 1)
     WTF_PLATFORM_IOS = 1
 else
     WTF_PLATFORM_IOS = 0
 endif
 
-ifeq ($(shell $(CC) -std=gnu++17 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_MAC ' | cut -d' ' -f3), 1)
+ifeq ($(shell $(CC) -std=gnu++14 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_MAC ' | cut -d' ' -f3), 1)
     WTF_PLATFORM_MAC = 1
 else
     WTF_PLATFORM_MAC = 0
 endif
 
-ifeq ($(shell $(CC) -std=gnu++17 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep USE_APPLE_INTERNAL_SDK | cut -d' ' -f3), 1)
+ifeq ($(shell $(CC) -std=gnu++14 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep USE_APPLE_INTERNAL_SDK | cut -d' ' -f3), 1)
     USE_APPLE_INTERNAL_SDK = 1
 else
     USE_APPLE_INTERNAL_SDK = 0
 endif
 
-ifeq ($(shell $(CC) -std=gnu++17 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ENABLE_ORIENTATION_EVENTS | cut -d' ' -f3), 1)
+ifeq ($(shell $(CC) -std=gnu++14 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ENABLE_ORIENTATION_EVENTS | cut -d' ' -f3), 1)
     ENABLE_ORIENTATION_EVENTS = 1
 endif
 
index 0ae57ff..01bdbae 100644 (file)
@@ -222,6 +222,7 @@ threedshadow
 window
 windowframe
 windowtext
+#if defined(WTF_PLATFORM_MAC) && WTF_PLATFORM_MAC
 -apple-system-header-text
 -apple-system-text-background
 -apple-system-control-background
@@ -243,6 +244,7 @@ windowtext
 -apple-system-quaternary-label
 -apple-system-grid
 -apple-system-separator
+#endif
 -apple-wireless-playback-target-active
 -apple-system-blue
 -apple-system-brown
index 364f1d3..01562f7 100644 (file)
@@ -1,3 +1,13 @@
+2018-06-27  Timothy Hatcher  <timothy@apple.com>
+
+        Don't expose new semantic -apple-system color keywords on iOS.
+        https://bugs.webkit.org/show_bug.cgi?id=187080
+        rdar://problem/41505699
+
+        Reviewed by Tim Horton.
+
+        * DerivedSources.make: Use gnu++14, since gnu++17 is giving errors on macOS 10.12.
+
 2018-06-27  Megan Gardner  <megan_gardner@apple.com>
 
         Fix IBeam issues with iPad apps on Mac
index f78d027..1e4cc51 100644 (file)
@@ -268,7 +268,7 @@ AUTOMATION_PROTOCOL_OUTPUT_FILES = \
 #
 
 ifeq ($(OS),MACOS)
-ifeq ($(shell $(CC) -std=gnu++17 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_IOS ' | cut -d' ' -f3), 1)
+ifeq ($(shell $(CC) -std=gnu++14 -x c++ -E -P -dM $(SDK_FLAGS) $(TARGET_TRIPLE_FLAGS) $(FRAMEWORK_FLAGS) $(HEADER_FLAGS) -include "wtf/Platform.h" /dev/null | grep ' WTF_PLATFORM_IOS ' | cut -d' ' -f3), 1)
        AUTOMATION_BACKEND_PLATFORM_ARGUMENTS = --platform iOS
 else
        AUTOMATION_BACKEND_PLATFORM_ARGUMENTS = --platform macOS