Remove ENABLE_OPENCL fully

[Don Olmstead]

There's still some ENABLE(OPENCL) code hanging out in the repository.

[Rob Buis]

Created attachment 353697 [details]
Patch

[Yusuke Suzuki]

Comment on attachment 353697 [details]
Patch

r=me. Nobody is using ENABLE(OPENCL). And we do not have implementations.

[WebKit Commit Bot]

Comment on attachment 353697 [details]
Patch

Clearing flags on attachment: 353697

Committed r237732: <https://trac.webkit.org/changeset/237732>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/45762732>

[Don Olmstead]

Comment on attachment 353697 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=353697&action=review

Beat me to the patch :)

Anyways I didn't throw it up because of the applyAll still being there. Seems like that was just there for OPENCL support so it should probably go as well.

> Source/WebCore/platform/graphics/filters/FilterEffect.h:96
>      inline void applyAll() { apply(); }

Should this still be here?

[Rob Buis]

Hi Don, right I took the gamble since you did not assign to yourself :)
The applyAll seems not needed, if you want to do a patch feel free, otherwise I can do it tomorrow.

[Rob Buis]

Reopening to attach new patch.

[Rob Buis]

Created attachment 353802 [details]
Patch

[Yusuke Suzuki]

Comment on attachment 353802 [details]
Patch

r=me

[WebKit Commit Bot]

Comment on attachment 353802 [details]
Patch

Clearing flags on attachment: 353802

Committed r237787: <https://trac.webkit.org/changeset/237787>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.