WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
246913
[web-animations] handle blending of layout-dependent <transform-list> values
https://bugs.webkit.org/show_bug.cgi?id=246913
Summary
[web-animations] handle blending of layout-dependent <transform-list> values
Antoine Quint
Reported
2022-10-23 03:02:01 PDT
We're failing a subtest in web-animations/interfaces/Animation/commitStyles.html: test(t => { const div = createDiv(t); div.style.width = '200px'; div.style.height = '200px'; const animation = div.animate({ transform: ["translate(100%, 0%)", "scale(3)"] }, 1000); animation.currentTime = 500; animation.commitStyles(); animation.cancel(); assert_equals(getComputedStyle(div).transform, "matrix(2, 0, 0, 2, 100, 0)", "Resolved transform is correct after commit."); }, 'Commits matrix-interpolated relative transforms'); Our current result is "matrix(2, 0, 0, 2, 0, 0)" because when we blend under commitStyles() layout has not happened yet for the div and the bounds passed to resolve the 100% translation value is 0,0.
Attachments
Add attachment
proposed patch, testcase, etc.
Antoine Quint
Comment 1
2022-10-23 03:08:19 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/5684
Antoine Quint
Comment 2
2022-10-24 01:33:10 PDT
Performing a layout should not be required to pass this test, I wrote all the details in
https://github.com/WebKit/WebKit/pull/5684#issuecomment-1288636644
.
Antoine Quint
Comment 3
2022-10-24 02:42:19 PDT
We'll need mix() support, I filed
https://bugs.webkit.org/show_bug.cgi?id=246937
. Retitling this bug to correctly identify the issue.
Radar WebKit Bug Importer
Comment 4
2022-10-30 03:02:17 PDT
<
rdar://problem/101725429
>
Ahmad Saleem
Comment 5
2024-03-05 18:35:05 PST
Still failing sub-test:
https://wpt.fyi/results/web-animations/interfaces/Animation/commitStyles.html?label=master&label=experimental&aligned=&q=safari%3Afail+firefox%3Apass+chrome%3Apass
Adding 'WPTImpact' and 'BrowserCompat' tags.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug