Bug 256136 – Features specific to other ports should not be compiled in

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

[Adrian Perez]

After bug #255518 I have noticed that some of the features which end
up in the built code do not make any sense for the GLib ports. For
example:

  AcceleratedDrawingEnabled:
    type: bool
    status: internal
    defaultsOverridable: true
    humanReadableName: "GraphicsLayerCA accelerated drawing"
    humanReadableDescription: "Enable GraphicsLayerCA accelerated drawing"
    ...

The above feature is specific to the Apple ports, as none of WPE, GTK,
WinCairo, or PlayStation can use CoreAnimation.

It would be good to go over the list in UnifiedWebPreferences.yaml and
add missing conditions for such cases; e.g. in the example above we need
to add:

    condition: USE(CA)

[Adrian Perez]

(In reply to Adrian Perez from comment #0)
> After bug #255518 I have noticed that some of the features which end
> up in the built code do not make any sense for the GLib ports. For
> example:
> 
>   AcceleratedDrawingEnabled:
>     type: bool
>     status: internal
>     defaultsOverridable: true
>     humanReadableName: "GraphicsLayerCA accelerated drawing"
>     humanReadableDescription: "Enable GraphicsLayerCA accelerated drawing"
>     ...
> 
> The above feature is specific to the Apple ports, as none of WPE, GTK,
> WinCairo, or PlayStation can use CoreAnimation.
> 
> It would be good to go over the list in UnifiedWebPreferences.yaml and
> add missing conditions for such cases; e.g. in the example above we need
> to add:
> 
>     condition: USE(CA)

Mmmmkay, maybe this wasn't the best example because the feature setting is
used also by ports that don't define USE(CA), so in this case the course of
action would be to change the description to read “GraphicsLayer accelerated
drawing” (without the “CA”); but the general idea applies to other feature
settings.

[Adrian Perez]

Pull request: https://github.com/WebKit/WebKit/pull/13392

[EWS]

Committed 263646@main (8cb6215d7346): <https://commits.webkit.org/263646@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/108857292>