color-gamut media query shouldn't ASSERT on invalid values
authordino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 23:20:39 +0000 (23:20 +0000)
committerdino@apple.com <dino@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Oct 2016 23:20:39 +0000 (23:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163303
<rdar://problem/28724566>

Reviewed by Simon Fraser.

Source/WebCore:

Remove the ASSERT_NOT_REACHED in the evaluation, since it
can be hit when there is a valid CSS keyword that is not
expected in the query.

Updated fast/media/mq-color-gamut.html

* css/MediaQueryEvaluator.cpp:
(WebCore::colorGamutEvaluate):

LayoutTests:

Add a 'none' test.

* fast/media/mq-color-gamut-expected.html:
* fast/media/mq-color-gamut.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/media/mq-color-gamut-expected.html
LayoutTests/fast/media/mq-color-gamut.html
Source/WebCore/ChangeLog
Source/WebCore/css/MediaQueryEvaluator.cpp

index 4d33d4e..621b672 100644 (file)
@@ -1,3 +1,16 @@
+2016-10-11  Dean Jackson  <dino@apple.com>
+
+        color-gamut media query shouldn't ASSERT on invalid values
+        https://bugs.webkit.org/show_bug.cgi?id=163303
+        <rdar://problem/28724566>
+
+        Reviewed by Simon Fraser.
+
+        Add a 'none' test.
+
+        * fast/media/mq-color-gamut-expected.html:
+        * fast/media/mq-color-gamut.html:
+
 2016-10-11  Chris Dumez  <cdumez@apple.com>
 
         Update MediaStream events to stop using legacy [ConstructorTemplate=Event]
index be1d070..2d81ef8 100644 (file)
@@ -4,11 +4,13 @@
 <style type="text/css">
 #a { background-color: green; color: white; }
 #b { background-color: green; color: white; }
+#c { background-color: green; color: white; }
 </style>
 </head>
 <body>
   <p id="a">This paragraph should have a green background.</p>
   <p id="b">This paragraph should have a blue background on a device with a display that supports something close to P3. Otherwise the background should be green.</p>
+  <p id="c">This paragraph should have a green background.</p>
   <p>Note that the expected results of this test assume a display that is not significantly wider than sRGB.</p>
 </body>
 </html>
index e7c2042..04f5aa2 100644 (file)
@@ -4,15 +4,20 @@
 <style type="text/css">
 #a { background-color: green; color: white; }
 #b { background-color: green; color: white; }
+#c { background-color: green; color: white; }
 
 @media (color-gamut: p3) {
 #b { background-color: blue; }
 }
+@media (color-gamut: none) {
+#b { background-color: blue; }
+}
 </style>
 </head>
 <body>
   <p id="a">This paragraph should have a green background.</p>
   <p id="b">This paragraph should have a blue background on a device with a display that supports something close to P3. Otherwise the background should be green.</p>
+  <p id="c">This paragraph should have a green background.</p>
   <p>Note that the expected results of this test assume a display that is not significantly wider than sRGB.</p>
 </body>
 </html>
index 31250d2..d5c6e4d 100644 (file)
@@ -1,3 +1,20 @@
+2016-10-11  Dean Jackson  <dino@apple.com>
+
+        color-gamut media query shouldn't ASSERT on invalid values
+        https://bugs.webkit.org/show_bug.cgi?id=163303
+        <rdar://problem/28724566>
+
+        Reviewed by Simon Fraser.
+
+        Remove the ASSERT_NOT_REACHED in the evaluation, since it
+        can be hit when there is a valid CSS keyword that is not
+        expected in the query.
+
+        Updated fast/media/mq-color-gamut.html
+
+        * css/MediaQueryEvaluator.cpp:
+        (WebCore::colorGamutEvaluate):
+
 2016-10-11  Chris Dumez  <cdumez@apple.com>
 
         Update MediaStream events to stop using legacy [ConstructorTemplate=Event]
index c6392be..75bb746 100644 (file)
@@ -255,7 +255,6 @@ static bool colorGamutEvaluate(CSSValue* value, const CSSToLengthConversionData&
         // FIXME: At some point we should start detecting displays that support more colors.
         return false;
     default:
-        ASSERT_NOT_REACHED();
         return true;
     }
 }