Standardize the `<meta name="color-scheme">` separator.
authortimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Apr 2019 23:55:47 +0000 (23:55 +0000)
committertimothy@apple.com <timothy@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 17 Apr 2019 23:55:47 +0000 (23:55 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193931

Reviewed by Simon Fraser.

Source/WebCore:

Drop support for comma as a valid seperator in <meta name="color-scheme"> to
match the proposal being tracked by: https://github.com/whatwg/html/issues/4504

Tests: css-dark-mode/color-scheme-meta.html
       css-dark-mode/older-syntax/supported-color-schemes-meta.html

* dom/Document.cpp:
(WebCore::isColorSchemeSeparator): Drop support for comma.

LayoutTests:

* css-dark-mode/color-scheme-meta-expected.txt:
* css-dark-mode/color-scheme-meta.html:
* css-dark-mode/older-syntax/supported-color-scheme-meta-expected.txt:
* css-dark-mode/older-syntax/supported-color-schemes-meta.html:

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

LayoutTests/ChangeLog
LayoutTests/css-dark-mode/color-scheme-meta-expected.txt
LayoutTests/css-dark-mode/color-scheme-meta.html
LayoutTests/css-dark-mode/older-syntax/supported-color-schemes-meta-expected.txt
LayoutTests/css-dark-mode/older-syntax/supported-color-schemes-meta.html
Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp

index b520cd4..8ae934a 100644 (file)
@@ -1,5 +1,17 @@
 2019-04-17  Timothy Hatcher  <timothy@apple.com>
 
+        Standardize the `<meta name="color-scheme">` separator.
+        https://bugs.webkit.org/show_bug.cgi?id=193931
+
+        Reviewed by Simon Fraser.
+
+        * css-dark-mode/color-scheme-meta-expected.txt:
+        * css-dark-mode/color-scheme-meta.html:
+        * css-dark-mode/older-syntax/supported-color-scheme-meta-expected.txt:
+        * css-dark-mode/older-syntax/supported-color-schemes-meta.html:
+
+2019-04-17  Timothy Hatcher  <timothy@apple.com>
+
         Rename `supported-color-schemes` to `color-scheme`.
         https://bugs.webkit.org/show_bug.cgi?id=197016
         rdar://problem/49980259
index afc914d..0491c80 100644 (file)
@@ -16,8 +16,8 @@ PASS Color schemes changed to light and a bogus value
 PASS Element colors are correct in dark color scheme with only light color scheme 
 PASS Color schemes changed to empty value 
 PASS Element colors are correct in dark color scheme with implicit light color scheme 
-PASS Color schemes changed to light and dark 
-PASS Element colors are correct in dark color scheme with light and dark color scheme 
+PASS Color schemes changed to light,dark 
+PASS Element colors are in light color scheme since comma is not an allowed seperator 
 PASS Color schemes changed to a bogus value and dark 
 PASS Element colors are correct in dark color scheme with dark color scheme 
 
index f52b75e..ce7aafe 100644 (file)
@@ -92,7 +92,7 @@ test(function() {
 }, "Element colors are correct in dark color scheme with only dark color scheme");
 
 test(function() {
-    document.getElementById("meta").content = "light ,foo ";
+    document.getElementById("meta").content = "light  foo ";
 }, "Color schemes changed to light and a bogus value");
 
 test(function() {
@@ -110,16 +110,16 @@ test(function() {
 }, "Element colors are correct in dark color scheme with implicit light color scheme");
 
 test(function() {
-    document.getElementById("meta").content = "   light, dark ";
-}, "Color schemes changed to light and dark");
+    document.getElementById("meta").content = "   light,dark ";
+}, "Color schemes changed to light,dark");
 
 test(function() {
-    // The semantic text color should be white now.
-    test_color_is_white("test1");
-}, "Element colors are correct in dark color scheme with light and dark color scheme");
+    // The semantic text color should be black now.
+    test_color_is_black("test1");
+}, "Element colors are in light color scheme since comma is not an allowed seperator");
 
 test(function() {
-    document.getElementById("meta").content = ", foo dark";
+    document.getElementById("meta").content = "  foo dark";
 }, "Color schemes changed to a bogus value and dark");
 
 test(function() {
index 2a6e63c..d3c7e68 100644 (file)
@@ -16,8 +16,8 @@ PASS Supported color schemes changed to light and a bogus value
 PASS Element colors are correct in dark color scheme with only light color scheme 
 PASS Supported color schemes changed to empty value 
 PASS Element colors are correct in dark color scheme with implicit light color scheme 
-PASS Supported color schemes changed to light and dark 
-PASS Element colors are correct in dark color scheme with light and dark color scheme 
+PASS Supported color schemes changed to light,dark 
+PASS Element colors are in light color scheme since comma is not an allowed seperator 
 PASS Supported color schemes changed to a bogus value and dark 
 PASS Element colors are correct in dark color scheme with dark color scheme 
 
index ea1f562..3284aea 100644 (file)
@@ -110,16 +110,16 @@ test(function() {
 }, "Element colors are correct in dark color scheme with implicit light color scheme");
 
 test(function() {
-    document.getElementById("meta").content = "   light, dark ";
-}, "Supported color schemes changed to light and dark");
+    document.getElementById("meta").content = "   light,dark ";
+}, "Supported color schemes changed to light,dark");
 
 test(function() {
-    // The semantic text color should be white now.
-    test_color_is_white("test1");
-}, "Element colors are correct in dark color scheme with light and dark color scheme");
+    // The semantic text color should be black now.
+    test_color_is_black("test1");
+}, "Element colors are in light color scheme since comma is not an allowed seperator");
 
 test(function() {
-    document.getElementById("meta").content = ", foo dark";
+    document.getElementById("meta").content = "  foo dark";
 }, "Supported color schemes changed to a bogus value and dark");
 
 test(function() {
index 2f14878..9d30a34 100644 (file)
@@ -1,5 +1,21 @@
 2019-04-17  Timothy Hatcher  <timothy@apple.com>
 
+        Standardize the `<meta name="color-scheme">` separator.
+        https://bugs.webkit.org/show_bug.cgi?id=193931
+
+        Reviewed by Simon Fraser.
+
+        Drop support for comma as a valid seperator in <meta name="color-scheme"> to
+        match the proposal being tracked by: https://github.com/whatwg/html/issues/4504
+
+        Tests: css-dark-mode/color-scheme-meta.html
+               css-dark-mode/older-syntax/supported-color-schemes-meta.html
+
+        * dom/Document.cpp:
+        (WebCore::isColorSchemeSeparator): Drop support for comma.
+
+2019-04-17  Timothy Hatcher  <timothy@apple.com>
+
         Rename `supported-color-schemes` to `color-scheme`.
         https://bugs.webkit.org/show_bug.cgi?id=197016
         rdar://problem/49980259
index cdfdaa2..4c31bbd 100644 (file)
@@ -3616,9 +3616,9 @@ void Document::updateViewportArguments()
 }
 
 #if ENABLE(DARK_MODE_CSS)
-static bool isColorSchemeSeparator(UChar character)
+static inline bool isColorSchemeSeparator(UChar character)
 {
-    return isASCIISpace(character) || character == ',';
+    return isASCIISpace(character);
 }
 
 static void processColorSchemeString(StringView colorScheme, const WTF::Function<void(StringView key)>& callback)