Bug 32224 – Turn on (SVG) Filters support, by default.

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

[Nikolas Zimmermann]

Enable (SVG) Filters support by default. The plan is to enable build on Mac only. The other platforms Skipped list will be adapted, to not run any test affected by the filter feature. If mac port builds fine, and the build bots are green again, the other platforms should be enabled one after another. Uploading a patch soon.

[Nikolas Zimmermann]

Created attachment 44418 [details]
Patch, excluding LayoutTests changes

Follow-up patch, will contain the LayoutTest results (big list of changes, all filter affected tests have to be rebaselined).

[WebKit Review Bot]

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

[Holger Freyther]

Comment on attachment 44418 [details]
Patch, excluding LayoutTests changes


> +        Turn on (SVG) Filters support, by default.
> +        https://bugs.webkit.org/show_bug.cgi?id=32224

rs=me from the impressions I got from the webkit-dev thread. According to Niko we do have enough auto tests for SVG filters.

[Nikolas Zimmermann]

Landed patch enabling mac build in r51800. Waiting for bot results before proceeding.

[Nikolas Zimmermann]

Comment on attachment 44418 [details]
Patch, excluding LayoutTests changes

Clearing review flag, this patch has been landed.

[Nikolas Zimmermann]

All mac build slaves work fine, w/o regression since turning on filters.
Qt has filters turned out, w/o layout test failures since r51841.
Now gtk...

[Nikolas Zimmermann]

(In reply to comment #6)
> All mac build slaves work fine, w/o regression since turning on filters.
> Qt has filters turned out, w/o layout test failures since r51841.
> Now gtk...
s/turned out/turned on/

[Nikolas Zimmermann]

Gtk + Filters work properly after r51847 (bots green).

[Nikolas Zimmermann]

Created attachment 44473 [details]
Build patches for win

Adding the win patches, so I can show them around before landing. Already got a blind rs=mjs, given that I'm willing to watch builds and fix :-)

[Dirk Schulze]

(In reply to comment #9)
> Created an attachment (id=44473) [details]
> Build patches for win
> 
> Adding the win patches, so I can show them around before landing. Already got a
> blind rs=mjs, given that I'm willing to watch builds and fix :-)

Filter are activated on every port. Can we close this bug now?

[Nikolas Zimmermann]

Agreed. Closing.