Bug 242498 – changing mask or filter property between two URLs on SVG element doesn't cause an update

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

[Cameron McCormack (:heycam)]

Caught by the tests I'm adding in bug 242497.

[Radar WebKit Bug Importer]

<rdar://problem/97058470>

[Said Abou-Hallawa]

Can you please attach a test case?

[Said Abou-Hallawa]

This will be fixed in the cocoa underlying frameworks.

[Said Abou-Hallawa]

Sorry I meant another bug to be closed.

[Cameron McCormack (:heycam)]

(In reply to Said Abou-Hallawa from comment #2)
> Can you please attach a test case?

The expected-fail LayoutTests/svg/resource-invalidation/mask-resource-invalidation.html and LayoutTests/svg/resource-invalidation/filter-resource-invalidation.html tests that I landed test this.

I have upcoming patches for lazy filter and mask resource invalidation that I'll upload soon.  The filter one fixes the filter invalidation issue, but the mask one doesn't.  I haven't looked into why that is yet.

[Takao Baba]

I'm not completely sure whether this is the same issue or new one, but there is a regression about this SVG filter issue on Safari 16.

Steps to reproduce:
Open https://jsbin.com/riyawenudu/edit?html,output

Expected behavior:
The rectangle color will be changed every second (red -> green -> blue -> red ...).

Actual behavior:
On safari 16 (both macOS and iOS), the color won't be changed.

I cannot create reduced testcase without using React.

[Cameron McCormack (:heycam)]

(In reply to Takao Baba from comment #6)
> I'm not completely sure whether this is the same issue or new one, but there
> is a regression about this SVG filter issue on Safari 16.
> 
> Steps to reproduce:
> Open https://jsbin.com/riyawenudu/edit?html,output

Thanks for reporting this issue with feComponentTransfer.  It's being fixed in bug 246602.

[Cameron McCormack (:heycam)]

Fixing the filter case in bug 247411.

[Karl Dubost]

This seems to be working for me on a recent build of Safari.

[Ahmad Saleem]

It is not reproducible on Safari Technology Preview 195 on Sonoma using test case from Comment 06 and it updates correctly each second to different color.

Marking this as 'Configuration Changed', if it is still reproducing, please reopen with newer test case.

[EWS]

Committed 283344@main (796a95b78ce1): <https://commits.webkit.org/283344@main>

Reviewed commits have been landed. Closing PR #33156 and removing active labels.