Skip to content
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

drop-shadow() filter is not updated in response to off-main-thread animations #2721

Closed
mstange opened this issue May 2, 2018 · 2 comments
Closed

Comments

@mstange
Copy link
Contributor

@mstange mstange commented May 2, 2018

https://bugzilla.mozilla.org/show_bug.cgi?id=1458661

Testcase 1
Testcase 2
Testcase 3 (works correctly but might still be worth confirming with once the bug is fixed)

In these testcases the black shadow lags behind. In the second testcase, the shadow gets squeezed and stretched in funny ways as the bounds of the filter input change.

@mstange
Copy link
Contributor Author

@mstange mstange commented May 3, 2018

So it looks like the shadow rendering is updated when the size of the input changes, but not when only the position of the input changes. And when the size of the input changes, the updated shadow is drawn into a rectangle that was computed based on the non-updated bounds.

gw3583 added a commit to gw3583/webrender that referenced this issue Jun 4, 2018
If the local rect of the content of a drop-shadow is changing
each frame, we also need to invalidate the GPU cache data for
the drop-shadow primitive.

Unfortunately we don't have a way in wrench right now to create
a regression test for this.

Fixes servo#2721.
@gw3583
Copy link
Collaborator

@gw3583 gw3583 commented Jun 4, 2018

Fix is #2799.

bors-servo added a commit that referenced this issue Jun 4, 2018
Invalidate drop-shadow data when local rect is animating.

If the local rect of the content of a drop-shadow is changing
each frame, we also need to invalidate the GPU cache data for
the drop-shadow primitive.

Unfortunately we don't have a way in wrench right now to create
a regression test for this.

Fixes #2721.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2799)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.