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
Fixed inner rectangle computation for transformed clips #2189
Conversation
@bors-servo r=glennw |
📌 Commit 6b4ddfb has been approved by |
Fixed inner rectangle computation for transformed clips <!-- 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/2189) <!-- Reviewable:end -->
☀️ Test successful - status-appveyor, status-travis |
@mrobinson This PR caused a talos-g1 failure. Panic stack and details in https://bugzilla.mozilla.org/show_bug.cgi?id=1423203#c10 onwards. |
@kvark Any idea about this one? I'll try to investigate a bit tomorrow. If it seems like the issue is quite tricky we can use my original (super conservative and inefficient) approach until we figure this one out, in order to not block the WR update in Gecko. |
@mrobinson yeah, I've been meaning to look into it but my Talos doesn't work and I got sucked into other things. Going with your original PR sounds fine to me, for now. |
Actually, upon another consideration that code is incorrect (non-concervative). We should definitely back it out. |
Okay. I have a workaround PR here #2205. |
This change is