[CSS Parser] Unskip media tests
authorhyatt@apple.com <hyatt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Dec 2016 03:22:36 +0000 (03:22 +0000)
committerhyatt@apple.com <hyatt@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 8 Dec 2016 03:22:36 +0000 (03:22 +0000)
https://bugs.webkit.org/show_bug.cgi?id=165580

Reviewed by Jon Lee.

The new parser will still create rules for hover/pointer queries with
unknown identifiers. Patch the tests to account for this.

* TestExpectations:
* fast/media/mq-any-hover-invalid-expected.txt:
* fast/media/mq-any-hover-invalid.html:
* fast/media/mq-any-pointer-invalid-expected.txt:
* fast/media/mq-any-pointer-invalid.html:
* fast/media/mq-hover-invalid-expected.txt:
* fast/media/mq-hover-invalid.html:
* fast/media/mq-pointer-invalid-expected.txt:
* fast/media/mq-pointer-invalid.html:

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

LayoutTests/ChangeLog
LayoutTests/TestExpectations
LayoutTests/fast/media/mq-any-hover-invalid-expected.txt
LayoutTests/fast/media/mq-any-hover-invalid.html
LayoutTests/fast/media/mq-any-pointer-invalid-expected.txt
LayoutTests/fast/media/mq-any-pointer-invalid.html
LayoutTests/fast/media/mq-hover-invalid-expected.txt
LayoutTests/fast/media/mq-hover-invalid.html
LayoutTests/fast/media/mq-pointer-invalid-expected.txt
LayoutTests/fast/media/mq-pointer-invalid.html

index 780919b..4ba4cfb 100644 (file)
@@ -1,3 +1,23 @@
+2016-12-07  Dave Hyatt  <hyatt@apple.com>
+
+        [CSS Parser] Unskip media tests
+        https://bugs.webkit.org/show_bug.cgi?id=165580
+
+        Reviewed by Jon Lee.
+
+        The new parser will still create rules for hover/pointer queries with
+        unknown identifiers. Patch the tests to account for this.
+
+        * TestExpectations:
+        * fast/media/mq-any-hover-invalid-expected.txt:
+        * fast/media/mq-any-hover-invalid.html:
+        * fast/media/mq-any-pointer-invalid-expected.txt:
+        * fast/media/mq-any-pointer-invalid.html:
+        * fast/media/mq-hover-invalid-expected.txt:
+        * fast/media/mq-hover-invalid.html:
+        * fast/media/mq-pointer-invalid-expected.txt:
+        * fast/media/mq-pointer-invalid.html:
+
 2016-12-07  Dean Jackson  <dino@apple.com>
 
         Remove runtime toggle for pointer-lock
index 68f6ffe..2148ca3 100644 (file)
@@ -1013,10 +1013,6 @@ webkit.org/b/165195 fast/css/remove-shorthand.html [ Pass Failure ]
 webkit.org/b/165195 fast/css/transform-inline-style.html [ Pass Failure ]
 webkit.org/b/165195 fast/css/uri-token-parsing.html [ Pass Failure ]
 webkit.org/b/165195 fast/dom/Window/getMatchedCSSRules-with-pseudo-elements-complex.html [ Pass Failure ]
-webkit.org/b/165195 fast/media/mq-any-hover-invalid.html [ Pass Failure ]
-webkit.org/b/165195 fast/media/mq-any-pointer-invalid.html [ Pass Failure ]
-webkit.org/b/165195 fast/media/mq-hover-invalid.html [ Pass Failure ]
-webkit.org/b/165195 fast/media/mq-pointer-invalid.html [ Pass Failure ]
 webkit.org/b/165195 fast/regions/region-styling/parsing-region-style-rule.html [ Pass Failure ]
 webkit.org/b/165195 fast/selectors/querySelector-pseudo-element.html [ Pass Failure ]
 webkit.org/b/165195 fast/selectors/querySelector-pseudo-element-inside-functional-pseudo-class-any.html [ Pass Failure ]
index f2d6eb8..c99d65c 100644 (file)
@@ -4,19 +4,19 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS window.matchMedia("(any-hover:)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: )").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: WebKit)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: fine)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: portrait)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: \"None\")").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-hover: 'None')").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS successfullyParsed is true
 
 TEST COMPLETE
index fc2fece..32025fb 100644 (file)
@@ -23,7 +23,7 @@
         shouldBeFalse('window.matchMedia("(' + testCase + ')").matches');
 
         targetStyle.innerText = "@media " + testCase + " { }";
-        shouldBe("document.styleSheets[1].cssRules[0].media.length", "0");
+        shouldBe("document.styleSheets[1].cssRules[0].media.length", "1");
     }
 </script>
 
index e400776..4bc18ae 100644 (file)
@@ -4,21 +4,21 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS window.matchMedia("(any-pointer:)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: )").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: WebKit)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: pointer)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: hover)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: portrait)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: \"None\")").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(any-pointer: 'None')").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 3cb1af9..2a974b6 100644 (file)
@@ -24,7 +24,7 @@
         shouldBeFalse('window.matchMedia("(' + testCase + ')").matches');
 
         targetStyle.innerText = "@media " + testCase + " { }";
-        shouldBe("document.styleSheets[1].cssRules[0].media.length", "0");
+        shouldBe("document.styleSheets[1].cssRules[0].media.length", "1");
     }
 </script>
 
index 17d369c..6d99400 100644 (file)
@@ -4,19 +4,19 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS window.matchMedia("(hover:)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: )").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: WebKit)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: fine)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: portrait)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: \"None\")").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(hover: 'None')").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS successfullyParsed is true
 
 TEST COMPLETE
index c724d10..101ee31 100644 (file)
@@ -23,7 +23,7 @@
         shouldBeFalse('window.matchMedia("(' + testCase + ')").matches');
 
         targetStyle.innerText = "@media " + testCase + " { }";
-        shouldBe("document.styleSheets[1].cssRules[0].media.length", "0");
+        shouldBe("document.styleSheets[1].cssRules[0].media.length", "1");
     }
 </script>
 
index 229c8b4..338c412 100644 (file)
@@ -4,21 +4,21 @@ On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE
 
 
 PASS window.matchMedia("(pointer:)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: )").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: WebKit)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: pointer)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: hover)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: portrait)").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: \"None\")").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS window.matchMedia("(pointer: 'None')").matches is false
-PASS document.styleSheets[1].cssRules[0].media.length is 0
+PASS document.styleSheets[1].cssRules[0].media.length is 1
 PASS successfullyParsed is true
 
 TEST COMPLETE
index 717e846..ad76c15 100644 (file)
@@ -24,7 +24,7 @@
         shouldBeFalse('window.matchMedia("(' + testCase + ')").matches');
 
         targetStyle.innerText = "@media " + testCase + " { }";
-        shouldBe("document.styleSheets[1].cssRules[0].media.length", "0");
+        shouldBe("document.styleSheets[1].cssRules[0].media.length", "1");
     }
 </script>