Bug 14010 – Add tests to check filter, mask, clip, opacity and marker ordering

• Status: RESOLVED
• Resolution: FIXED
• Component: SVG

[Eric Seidel (no email)]

Add tests to check filter, mask, clip, opacity and marker ordering

I just added a prepareToRenderSVGContent to simplify handling of filters, masks and clips throughout all of the various SVG renderers.  In doing so, I realized that we were ordering filters slightly differently in the different classes.  I'm not yet 100% sure which ordering is correct, but we definitely need to add some tests to make sure that we keep filter, mask, clip, opacity and marker operations in the correct order going forward.

This just requires someone to write up a nice test case. :)

[Dirk Schulze]

We realy use the wrong order. At first the filter should be applied to the object, than masker/clipping and opacity at the end. The order of masker and clipper doesn't realy matter. Either we clip the content away and mask afterwards or we mask the content and clip it afterwards. There is no difference in the result.
I'll upload a patch.

[Dirk Schulze]

Created attachment 45139 [details]
Correct order of filter clipper and masker

Correct order of filter clipper and masker.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45139 [details] without any errors.

[Eric Seidel (no email)]

Comment on attachment 45139 [details]
Correct order of filter clipper and masker

Ideally your ChangeLog should link to the relevant section of the spec.  Or at least the bug should.

It looks like the rect in the mask you made doesn't do anything.  It's not clear to me from the example how it would have changed from the old behavior ro the new one, but I believe you when you say it changed.

[Dirk Schulze]

Created attachment 45181 [details]
Correct order of filter clipper and masker

The mask on the example just dims the contrast and shows, that it doesn't have any affect on the clipping. You can still see, that the clipper affects on top of the filtering.
I changed the Changelog and mention the part of the spec. I swiched the masker and clipper. We might profit of clipping away some context before the masking process later.

[WebKit Review Bot]

style-queue ran check-webkit-style on attachment 45181 [details] without any errors.

[Nikolas Zimmermann]

Comment on attachment 45181 [details]
Correct order of filter clipper and masker

krit++ Finally! r=me.

[WebKit Commit Bot]

Comment on attachment 45181 [details]
Correct order of filter clipper and masker

Clearing flags on attachment: 45181

Committed r52394: <http://trac.webkit.org/changeset/52394>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.