Bug 144193 – Rename WTF_USE_3D_GRAPHICS to ENABLE_GRAPHICS_CONTEXT_3D

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Michael Catanzaro]

Rename WTF_USE_3D_GRAPHICS to ENABLE_GRAPHICS_CONTEXT_3D

See:

https://bugs.webkit.org/show_bug.cgi?id=144105#c5
https://lists.webkit.org/pipermail/webkit-dev/2015-April/027404.html

[Michael Catanzaro]

Created attachment 251640 [details]
Patch

[Michael Catanzaro]

Created attachment 251642 [details]
Patch

[Martin Robinson]

Comment on attachment 251642 [details]
Patch

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

> Source/cmake/OptionsEfl.cmake:92
> +WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_GRAPHICS_CONTEXT_3D PUBLIC ON)

I think this should always be private. It will always be the result of other options. In the case of EFL and GTK+, it's equivalent to texture_mapper_enabled || webgl_enabled. perhaps it shouldn't even be an option, just a variable that is set when certain options are present.

[Michael Catanzaro]

(In reply to comment #3)
> I think this should always be private.

Yes, but EFL doesn't use private options yet (since I didn't want to pick and choose for them which to be private and which public).

> It will always be the result of other
> options. In the case of EFL and GTK+, it's equivalent to
> texture_mapper_enabled || webgl_enabled. perhaps it shouldn't even be an
> option, just a variable that is set when certain options are present.

OK, I agree. But if it's not an option, I'm not sure it should be named with ENABLE.

[Martin Robinson]

(In reply to comment #4)
> OK, I agree. But if it's not an option, I'm not sure it should be named with
> ENABLE.

I think ENABLE is okay even if it's not an option, but I'm open to being convinced otherwise. :)

[Michael Catanzaro]

Created attachment 251777 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 251777 [details]
Patch

Clearing flags on attachment: 251777

Committed r183432: <http://trac.webkit.org/changeset/183432>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.