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

First pass of GPU clipping support. #131

Merged
merged 1 commit into from Jan 8, 2016
Merged

First pass of GPU clipping support. #131

merged 1 commit into from Jan 8, 2016

Conversation

@glennw
Copy link
Member

glennw commented Jan 7, 2016

  • Add support for clip in/out rects on GPU.
  • Remove CPU polygon clipping code.
  • Fix all known vertex cracks and pixel sparkling.

There is still some code cleanup to do, and missing features, such as:

  • Nested complex clip regions.
  • Switching angle gradients to rectangles.
  • Complex clip regions + border radius.

These should be fairly simple to implement on top of these changes.

@glennw
Copy link
Member Author

glennw commented Jan 7, 2016

@pcwalton
Copy link
Collaborator

pcwalton commented Jan 8, 2016

I get thread '<unnamed>' panicked at 'assertion failed: border_radius == 0.0', /Users/pcwalton/Source/servo-packages/webrender/src/batch_builder.rs:911 on http://github.com/servo/servo.

* Add support for clip in/out rects on GPU.
* Remove CPU polygon clipping code.
* Fix all known vertex cracks and pixel sparkling.

There is still some code cleanup to do, and missing features, such as:
 * Nested complex clip regions.
 * Switching angle gradients to rectangles.
 * Complex clip regions + border radius.

These *should* be fairly simple to implement on top of these changes.
@glennw glennw force-pushed the gpu-clip branch from b51fc37 to 89aa49a Jan 8, 2016
@glennw
Copy link
Member Author

glennw commented Jan 8, 2016

@pcwalton That assert is because I haven't done the clip support for box-shadow + border-radius yet. I was going to do that as a follow up PR, so I've commented that assert out for now, if you're ok with doing that as a follow up?

glennw added a commit that referenced this pull request Jan 8, 2016
First pass of GPU clipping support.
@glennw glennw merged commit 88cc763 into master Jan 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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