mirror of
https://github.com/servo/servo.git
synced 2025-08-05 13:40:08 +01:00
Auto merge of #29789 - Loirooriol:do-not-skip-css-variables-tests, r=mrobinson
Stop skipping css-variables tests in layout-2020 <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [ ] These changes fix #___ (GitHub issue number if applicable) <!-- Either: --> - [X] There are tests for these changes OR - [ ] These changes do not require tests because ___ <!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.--> <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
This commit is contained in:
commit
032cf64d1b
14 changed files with 130 additions and 81 deletions
|
@ -32,6 +32,8 @@ skip: true
|
|||
skip: false
|
||||
[css-ui]
|
||||
skip: false
|
||||
[css-variables]
|
||||
skip: false
|
||||
[cssom]
|
||||
skip: false
|
||||
[cssom-view]
|
||||
|
|
|
@ -1,28 +0,0 @@
|
|||
[css-variable-change-style-001.html]
|
||||
[Test changing 'color' value to become a css variable]
|
||||
expected: FAIL
|
||||
|
||||
[Test declaration changes on 'color' as variable]
|
||||
expected: FAIL
|
||||
|
||||
[Avoid masking differences on 'color' due to declaration changes]
|
||||
expected: FAIL
|
||||
|
||||
[Test declaration changes on 'white-space' as variable]
|
||||
expected: FAIL
|
||||
|
||||
[Test declaration changes on 'background-color' as variable]
|
||||
expected: FAIL
|
||||
|
||||
[Test changing 'white-space' value to become a css variable]
|
||||
expected: FAIL
|
||||
|
||||
[Test changing 'background-color' value to become a css variable]
|
||||
expected: FAIL
|
||||
|
||||
[Avoid masking differences on 'background-color' due to declaration changes]
|
||||
expected: FAIL
|
||||
|
||||
[Avoid masking differences on 'white-space' due to declaration changes]
|
||||
expected: FAIL
|
||||
|
|
@ -1,10 +0,0 @@
|
|||
[css-variable-change-style-002.html]
|
||||
[Declaration changes on 'color' propagate to all variable references]
|
||||
expected: FAIL
|
||||
|
||||
[Declaration changes on 'background-color' propagate to all variable references]
|
||||
expected: FAIL
|
||||
|
||||
[Declaration changes on 'white-space' propagate to all variable references]
|
||||
expected: FAIL
|
||||
|
|
@ -0,0 +1,15 @@
|
|||
[variable-animation-from-to.html]
|
||||
bug: https://github.com/servo/servo/issues/26625
|
||||
expected: TIMEOUT
|
||||
[Verify CSS variable value before animation]
|
||||
expected: FAIL
|
||||
|
||||
[Verify substituted color value before animation]
|
||||
expected: FAIL
|
||||
|
||||
[Verify CSS variable value after animation]
|
||||
expected: NOTRUN
|
||||
|
||||
[Verify substituted color value after animation]
|
||||
expected: NOTRUN
|
||||
|
|
@ -0,0 +1,15 @@
|
|||
[variable-animation-over-transition.html]
|
||||
bug: https://github.com/servo/servo/issues/26625
|
||||
expected: TIMEOUT
|
||||
[Verify CSS variable value before animation]
|
||||
expected: FAIL
|
||||
|
||||
[Verify substituted color value before animation]
|
||||
expected: FAIL
|
||||
|
||||
[Verify CSS variable value after animation]
|
||||
expected: NOTRUN
|
||||
|
||||
[Verify substituted color value after animation]
|
||||
expected: NOTRUN
|
||||
|
|
@ -0,0 +1,8 @@
|
|||
[variable-animation-substitute-into-keyframe-transform.html]
|
||||
bug: https://github.com/servo/servo/issues/26625
|
||||
[Verify transform before animation]
|
||||
expected: FAIL
|
||||
|
||||
[Verify transform after animation]
|
||||
expected: FAIL
|
||||
|
|
@ -0,0 +1,6 @@
|
|||
[variable-animation-to-only.html]
|
||||
bug: https://github.com/servo/servo/issues/26625
|
||||
expected: TIMEOUT
|
||||
[Verify CSS variable value after animation]
|
||||
expected: TIMEOUT
|
||||
|
|
@ -1,33 +0,0 @@
|
|||
[variable-cycles.html]
|
||||
[Self-cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Simple a/b cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Three-var cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle that starts in the middle of a chain]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with extra edge]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with extra edge (2)]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with extra edge (3)]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with secondary cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with overlapping secondary cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle with deeper secondary cycle]
|
||||
expected: FAIL
|
||||
|
||||
[Cycle via fallback]
|
||||
expected: FAIL
|
|
@ -1,4 +1,10 @@
|
|||
[variable-definition.html]
|
||||
[CSSOM.setProperty with space 1]
|
||||
expected: FAIL
|
||||
|
||||
[CSSOM.setProperty with space 2]
|
||||
expected: FAIL
|
||||
|
||||
[no char variable]
|
||||
expected: FAIL
|
||||
|
||||
|
@ -62,12 +68,6 @@
|
|||
[ leading white space (double space) 2 (Cascading)]
|
||||
expected: FAIL
|
||||
|
||||
[CSSOM.setProperty with space 1]
|
||||
expected: FAIL
|
||||
|
||||
[CSSOM.setProperty with space 2]
|
||||
expected: FAIL
|
||||
|
||||
[can overwrite with no value]
|
||||
expected: FAIL
|
||||
|
||||
|
@ -76,3 +76,39 @@
|
|||
|
||||
[can overwrite with no value (Cascading)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (single space)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (double space) 2]
|
||||
expected: FAIL
|
||||
|
||||
[!important]
|
||||
expected: FAIL
|
||||
|
||||
[!important (with space)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (single space) (Computed Style)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (double space) 2 (Computed Style)]
|
||||
expected: FAIL
|
||||
|
||||
[!important (Computed Style)]
|
||||
expected: FAIL
|
||||
|
||||
[!important (with space) (Computed Style)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (single space) (Cascading)]
|
||||
expected: FAIL
|
||||
|
||||
[ trailing white space (double space) 2 (Cascading)]
|
||||
expected: FAIL
|
||||
|
||||
[!important (Cascading)]
|
||||
expected: FAIL
|
||||
|
||||
[!important (with space) (Cascading)]
|
||||
expected: FAIL
|
||||
|
|
|
@ -0,0 +1,13 @@
|
|||
[variable-first-letter.html]
|
||||
[color]
|
||||
expected: FAIL
|
||||
|
||||
[font-size]
|
||||
expected: FAIL
|
||||
|
||||
[font-weight]
|
||||
expected: FAIL
|
||||
|
||||
[nested color]
|
||||
expected: FAIL
|
||||
|
|
@ -0,0 +1,13 @@
|
|||
[variable-first-line.html]
|
||||
[color]
|
||||
expected: FAIL
|
||||
|
||||
[font-size]
|
||||
expected: FAIL
|
||||
|
||||
[font-weight]
|
||||
expected: FAIL
|
||||
|
||||
[nested color]
|
||||
expected: FAIL
|
||||
|
|
@ -0,0 +1,13 @@
|
|||
[variable-invalidation.html]
|
||||
[css rule test]
|
||||
expected: FAIL
|
||||
|
||||
[css rule test important]
|
||||
expected: FAIL
|
||||
|
||||
[inline style test]
|
||||
expected: FAIL
|
||||
|
||||
[inline style test important]
|
||||
expected: FAIL
|
||||
|
|
@ -0,0 +1,3 @@
|
|||
[variable-reference-without-whitespace.html]
|
||||
type: reftest
|
||||
expected: FAIL
|
|
@ -8,12 +8,8 @@
|
|||
[A custom property referencing a non-existent variable is treated as unset]
|
||||
expected: FAIL
|
||||
|
||||
[Custom properties in a cycle become guaranteed-invalid]
|
||||
expected: FAIL
|
||||
|
||||
[A custom property referencing a cycle becomes guaranteed-invalid]
|
||||
expected: FAIL
|
||||
|
||||
[A custom property referencing a non-existent variable becomes guaranteed-invalid]
|
||||
expected: FAIL
|
||||
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue