Bug 179009 – [WPE] Clean up OptionsWPE.cmake

• Status: RESOLVED
• Resolution: FIXED
• Component: WPE WebKit

[Michael Catanzaro]

Clean up OptionsWPE.cmake. Some of the public options look like things we might not want to expose, and there's unneeded code to support disabling options that are not public.

[Michael Catanzaro]

Zan, as part of this change, I'm auditing the public options list. I'd like to expose only options that we actually support toggling on or off, like we do for GTK. I'm guessing that the following public options should be removed:

ENABLE_3D_TRANSFORMS (should be always on)
ENABLE_CSS_SELECTORS_LEVEL4 (should be always on)
ENABLE_DEVICE_ORIENTATION (should be always off)
ENABLE_GEOLOCATION (should be always off)
ENABLE_MEDIA_CONTROLS_SCRIPT (should be always on, and probably removed as all ports use it now)
ENABLE_MEDIA_SOURCE (should be always off until Alicia is ready to turn it on, then always on)

I don't know about ENABLE_ACCELERATED_2D_CANVAS... what do you think? It is public on GTK, since apparently it's useful for performance on embedded devices. But since it's off by default, do you know if WPE has ever been tested in this configuration? We should probably leave it private unless we know someone is using it, I guess?

What about ENABLE_WEB_CRYPTO? I assumed that you would want it always on, especially since the GCrypt dependency is required for CryptoDigest anyway, so it's not possible to avoid that dependency. Can I make it private?

And how about ENABLE_WEBGL? Would anyone really ever want to turn that off? Can we hide it too?

[Michael Catanzaro]

Decision is to do as suggested above, except ENABLE_ACCELERATED_2D_CANVAS to remain public, and ENABLE_MEDIA_SOURCE to become private and ON by default.

[Michael Catanzaro]

Created attachment 325527 [details]
Patch

[Michael Catanzaro]

Committed r224492: <https://trac.webkit.org/changeset/224492>

[Michael Catanzaro]

Committed r224493: <https://trac.webkit.org/changeset/224493>