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

gfx: Simplify complex clipping regions as we construct them. #10331

Merged

Conversation

pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Apr 1, 2016

This allows WebRender to correctly render complex clipping regions that
can be reduced to single rounded rectangles. WebRender still can't
render rounded rectangles with arbitrary intersections yet, but this
allows it to handle many more cases.

Closes servo/webrender#241.

r? @glennw


This change is Reviewable

@highfive
Copy link

highfive commented Apr 1, 2016

Heads up! This PR modifies the following files:

  • @jgraham: tests/wpt/mozilla/meta/MANIFEST.json, tests/wpt/mozilla/tests/css/border_radius_in_border_radius_a.html, tests/wpt/mozilla/tests/css/border_radius_in_border_radius_ref.html

@highfive highfive added the S-awaiting-review There is new code that needs to be reviewed. label Apr 1, 2016
@pcwalton
Copy link
Contributor Author

pcwalton commented Apr 1, 2016

@paulrouget This is not a complete fix, but it fixes your test case. Let me know if you need to handle more cases than this right now.

This allows WebRender to correctly render complex clipping regions that
can be reduced to single rounded rectangles. WebRender still can't
render rounded rectangles with arbitrary intersections yet, but this
allows it to handle many more cases.

Closes servo/webrender#241.
@glennw
Copy link
Member

glennw commented Apr 1, 2016

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 22a1c11 has been approved by glennw

@highfive highfive added S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. and removed S-awaiting-review There is new code that needs to be reviewed. labels Apr 1, 2016
@paulrouget
Copy link
Contributor

@paulrouget This is not a complete fix, but it fixes your test case. Let me know if you need to handle more cases than this right now.

Tested. It fixes our issues in browserhtml. So that's enough. Thank you.

@bors-servo
Copy link
Contributor

⌛ Testing commit 22a1c11 with merge 4e21517...

bors-servo pushed a commit that referenced this pull request Apr 1, 2016
…glennw

gfx: Simplify complex clipping regions as we construct them.

This allows WebRender to correctly render complex clipping regions that
can be reduced to single rounded rectangles. WebRender still can't
render rounded rectangles with arbitrary intersections yet, but this
allows it to handle many more cases.

Closes servo/webrender#241.

r? @glennw

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10331)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - android, arm32, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, status-appveyor

@bors-servo bors-servo merged commit 22a1c11 into servo:master Apr 2, 2016
@highfive highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Apr 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants