New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Initial support for WorkletAnimation.playbackRate #14946
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already reviewed downstream.
daa59a2
to
2deae27
Compare
2deae27
to
4d36b26
Compare
4d36b26
to
f8dd7f2
Compare
Changes for scroll-linked worklet animations will come in a separate pull request. Bug: 852475 Change-Id: Ie6dd14242797cbc14a7e1e377f23c3dda174fe15 Reviewed-on: https://chromium-review.googlesource.com/c/1423537 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Reviewed-by: Majid Valipour <majidvp@chromium.org> Commit-Queue: Olga Gerchikov <gerchiko@microsoft.com> Cr-Commit-Position: refs/heads/master@{#628800}
f8dd7f2
to
59f1c95
Compare
Taskcluster error seems to be transient (complaining about not being able to find the window), so rebasing + force-pushing to retry. |
Close this PR because the Chromium CL does not have exportable changes. |
Taskcluster found this flakiness in Chrome Dev:
@stephenmcgruer @majido do you think this is a flaw with the test, or a Chrome bug? If you know Olga's GitHub handle also please poke her here. Thanks! |
We have noticed that the animation worklet tests have become flaky when composited animations are enabled. I believe wpt is running the tests under regular chrome (as opposed to content_shell) so animations are composited. I am actively investigating this. It is tracked here on Chromium side: https://bugs.chromium.org/p/chromium/issues/detail?id=930462 |
@foolip per my earlier comment, I think this is an issue that is affecting multiple tests. So I suggest merging this test while I investigate the flakiness. It may be a chrome bug or even if the tests are problematic we have to update them all. So nothing special about this test. |
Thanks @majido for the link, hopefully we'll soon have the ability to link failures to bugs and then we can link to https://bugs.chromium.org/p/chromium/issues/detail?id=930462 for this. I'll go ahead and admin merge. |
Changes for scroll-linked worklet animations will come in a separate pull request.
Bug: 852475
Change-Id: Ie6dd14242797cbc14a7e1e377f23c3dda174fe15
Reviewed-on: https://chromium-review.googlesource.com/c/1423537
Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Majid Valipour <majidvp@chromium.org>
Commit-Queue: Olga Gerchikov <gerchiko@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#628800}