Skip to content
This repository has been archived by the owner. It is now read-only.

Implement a `FillWithBlur` method for `box-shadow` #115

Closed
wants to merge 1 commit into from

Conversation

@pcwalton
Copy link
Contributor

pcwalton commented Nov 17, 2014

This seems to allow us to achieve the correct behavior cleanly. I've only implemented this in the Skia backend for now, where it ought to be GPU accelerated if Skia supports that (via SkShader).

r? @SimonSapin

@metajack
Copy link
Contributor

metajack commented Nov 17, 2014

Should this go upstream?

@pcwalton
Copy link
Contributor Author

pcwalton commented Nov 17, 2014

Yes.

@pcwalton pcwalton force-pushed the pcwalton:box-shadow branch from 908f28d to a12f388 Nov 18, 2014
@pcwalton
Copy link
Contributor Author

pcwalton commented Nov 18, 2014

r? @SimonSapin/@metajack

This is a revised version that does box shadows more or less exactly how Chromium does.

@pcwalton
Copy link
Contributor Author

pcwalton commented Nov 18, 2014

Closing because the filter pipeline stuff in upstream, more recent Azure will probably obviate the need for this.

@pcwalton pcwalton closed this Nov 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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