Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: c710b3e74570dd69487ac9b9a72ebc1ffb0f80f3
https://github.com/WebKit/WebKit/commit/c710b3e74570dd69487ac9b9a72ebc1ffb0f80f3
Author: Darin Adler <[email protected]>
Date: 2025-06-09 (Mon, 09 Jun 2025)
Changed paths:
M LayoutTests/TestExpectations
M LayoutTests/animations/resources/animation-test-helpers.js
M LayoutTests/fast/css/CSSStyleDeclaration-parameters-expected.txt
M LayoutTests/fast/css/CSSStyleDeclaration-parameters.html
M LayoutTests/fast/css/font-systemFontID-parsing-expected.txt
M LayoutTests/fast/css/font-systemFontID-parsing.html
M LayoutTests/fast/css/getComputedStyle/computed-style-font-expected.txt
M LayoutTests/fast/css/getComputedStyle/computed-style-font.html
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-position-expected.txt
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-position.html
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-shorthand-expected.txt
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-shorthand.html
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-size-expected.txt
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-size.html
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-border-color-shorthand-expected.txt
M
LayoutTests/fast/css/getComputedStyle/getComputedStyle-border-color-shorthand.html
M
LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-roundtrip-expected.txt
M
LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-roundtrip.html
M LayoutTests/fast/css3-text/font-synthesis-parse-expected.txt
M LayoutTests/fast/css3-text/font-synthesis-parse.html
R
LayoutTests/platform/glib/transitions/svg-text-shadow-transition-expected.txt
R
LayoutTests/platform/gtk/transitions/svg-text-shadow-transition-expected.png
R
LayoutTests/platform/ios/transitions/svg-text-shadow-transition-expected.txt
R
LayoutTests/platform/mac/transitions/svg-text-shadow-transition-expected.png
R
LayoutTests/platform/mac/transitions/svg-text-shadow-transition-expected.txt
R
LayoutTests/platform/win/transitions/svg-text-shadow-transition-expected.txt
M LayoutTests/transitions/frames-timing-function-expected.txt
M LayoutTests/transitions/lengthsize-transition-to-from-auto-expected.txt
M LayoutTests/transitions/lengthsize-transition-to-from-auto.html
M LayoutTests/transitions/resources/transition-test-helpers.js
A LayoutTests/transitions/svg-text-shadow-transition-expected.txt
M LayoutTests/transitions/svg-text-shadow-transition.html
M LayoutTests/transitions/transition-to-from-auto.html
M LayoutTests/transitions/transition-to-from-undefined.html
Log Message:
-----------
Eliminate use of getPropertyCSSValue from tests, part 2
https://bugs.webkit.org/show_bug.cgi?id=293590
rdar://152048914
Reviewed by Anne van Kesteren and Sam Weinig.
Some of this is removing possibly-useful test coverage; we should be cautious
about that.
See comments in https://github.com/WebKit/WebKit/pull/45904
* LayoutTests/TestExpectations: Expect
transitions/svg-text-shadow-transition.html to pass.
There is some risk that it will now be flaky, and we'll have to add another
flakiness expectation like the ones from webkit.org/b/137883.
* LayoutTests/animations/resources/animation-test-helpers.js:
(getPropertyValue): Rewrote this to have fewer special cases and not use
getPropertyCSSValue at all.
* LayoutTests/fast/css/CSSStyleDeclaration-parameters-expected.txt: Updated.
* LayoutTests/fast/css/CSSStyleDeclaration-parameters.html: Test
getPropertyValue
instead of getPropertyCSSValue.
* LayoutTests/fast/css/font-systemFontID-parsing-expected.txt: Updated.
* LayoutTests/fast/css/font-systemFontID-parsing.html: Test getPropertyValue
instead
of getPropertyCSSValue. Use js-test.js.
* LayoutTests/fast/css/getComputedStyle/computed-style-font-expected.txt:
Updated.
* LayoutTests/fast/css/getComputedStyle/computed-style-font.html: Removed the
tests of
getPropertyCSSValue. Simplified the tests that were left behind.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-position-expected.txt:
Updated.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-position.html:
Removed the test of getPropertyCSSValue.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-shorthand-expected.txt:
Updated.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-shorthand.html:
Removed the tests of getPropertyCSSValue. Simplified the tests that were left
behind.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-size-expected.txt:
Updated.
* LayoutTests/fast/css/getComputedStyle/getComputedStyle-background-size.html:
Removed the tests of getPropertyCSSValue. Simplified the tests that were left
behind.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-border-color-shorthand-expected.txt:
Updated.
*
LayoutTests/fast/css/getComputedStyle/getComputedStyle-border-color-shorthand.html:
Removed the tests of getPropertyCSSValue. Simplified the tests that were left
behind.
*
LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-roundtrip-expected.txt:
Updated.
*
LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-skip/text-decoration-skip-roundtrip.html:
Test with getPropertyValue instead of getPropertyCSSValue.
* LayoutTests/fast/css3-text/font-synthesis-parse-expected.txt: Updated.
* LayoutTests/fast/css3-text/font-synthesis-parse.html: Test with
getPropertyValue
instead of getPropertyCSSValue.
*
LayoutTests/platform/glib/transitions/svg-text-shadow-transition-expected.txt:
Removed.
* LayoutTests/platform/gtk/transitions/svg-text-shadow-transition-expected.png:
Removed.
* LayoutTests/platform/ios/transitions/svg-text-shadow-transition-expected.txt:
Removed.
* LayoutTests/platform/mac/transitions/svg-text-shadow-transition-expected.png:
Removed.
* LayoutTests/platform/mac/transitions/svg-text-shadow-transition-expected.txt:
Removed.
* LayoutTests/platform/win/transitions/svg-text-shadow-transition-expected.txt:
Removed.
* LayoutTests/transitions/frames-timing-function-expected.txt: Updated expected
failure messages since the test helper now helpfully cites the single value
instead
of the whole matrix.
* LayoutTests/transitions/lengthsize-transition-to-from-auto-expected.txt:
Updated.
* LayoutTests/transitions/lengthsize-transition-to-from-auto.html: Added an
expected
value of "auto" since the test helper no longer ignores its presence.
* LayoutTests/transitions/resources/transition-test-helpers.js: Rewrote the
checking
code to use the expected value types to decide what kind of comparision to do
and use
the string computed style value rather than using the result of
getPropertyCSSValue.
* LayoutTests/transitions/svg-text-shadow-transition-expected.txt: Added.
This is now a text based text rather than a pixel test.
* LayoutTests/transitions/svg-text-shadow-transition.html: Change expectations
to
match the serialization of text-shadow, including the color.
* LayoutTests/transitions/transition-to-from-auto.html: Expect the number 50
rather
than the string "50" now that the test helper uses the expected type to figure
out
how to check values. Alternatively could have expected the string "50px", but
this
preserves the old behavior and the tolerance.
* LayoutTests/transitions/transition-to-from-undefined.html: Expect the numbers
100
and 50 rather than the strings "100" and "50" now that the test helper uses the
expected
type to figure out how to check values. Alternatively could have expected the
strings
"100px" and "50px", but this preserves the old behavior and the tolerance.
Canonical link: https://commits.webkit.org/295994@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes