Bug 232618 - Add a runtime flag to enable CSS Transforms Level 2 spec compliant behavior
Summary: Add a runtime flag to enable CSS Transforms Level 2 spec compliant behavior
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Martin Robinson
URL:
Keywords: InRadar
Depends on:
Blocks: 229141 232619
  Show dependency treegraph
 
Reported: 2021-11-02 04:57 PDT by Martin Robinson
Modified: 2021-12-01 02:53 PST (History)
2 users (show)

See Also:


Attachments
Patch (3.78 KB, patch)
2021-11-04 11:29 PDT, Martin Robinson
no flags Details | Formatted Diff | Diff
Patch (1.53 KB, patch)
2021-11-30 05:18 PST, Martin Robinson
no flags Details | Formatted Diff | Diff
Patch (1.52 KB, patch)
2021-12-01 02:08 PST, Martin Robinson
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Robinson 2021-11-02 04:57:46 PDT
Some aspects of the CSS Transforms Level 2 specification break backward compatibility. Having a runtime flag for this behavior will allowing experimenting with turning it on in order to test compatibility.
Comment 1 Martin Robinson 2021-11-04 11:29:02 PDT
Created attachment 443325 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2021-11-09 03:58:22 PST
<rdar://problem/85196916>
Comment 3 Simon Fraser (smfr) 2021-11-29 08:41:01 PST
Comment on attachment 443325 [details]
Patch

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

r- to switch to a setting.

> Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml:1325
> +  humanReadableDescription: "Enable Transforms Level 2 spec compliant behavior that is backwards incompatble"

"incompatble".

I think we can come up with a better name/description. Maybe "CSS 3D Transform Interoperability"?

> Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml:1326
> +  webcoreBinding: RuntimeEnabledFeatures

We don't use RuntimeEnabledFeatures any more.
Comment 4 Martin Robinson 2021-11-30 05:18:46 PST
Created attachment 445409 [details]
Patch
Comment 5 Martin Robinson 2021-11-30 05:33:52 PST
(In reply to Simon Fraser (smfr) from comment #3)
> Comment on attachment 443325 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=443325&action=review
> 
> r- to switch to a setting.

Thanks for the review! I've uploaded a new version of the change that addresses your comments.


> > Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml:1325
> > +  humanReadableDescription: "Enable Transforms Level 2 spec compliant behavior that is backwards incompatble"
> 
> "incompatble".
> 
> I think we can come up with a better name/description. Maybe "CSS 3D
> Transform Interoperability"?

I've improved the description and the naming along the lines you suggested.

> 
> > Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml:1326
> > +  webcoreBinding: RuntimeEnabledFeatures
> 
> We don't use RuntimeEnabledFeatures any more.

Ah, I wasn't aware of this! I've converted this to a setting. It will require piping things through a bit (since it seems that settings are accessed through the Document), but this isn't a bit deal.
Comment 6 Simon Fraser (smfr) 2021-11-30 09:47:00 PST
Comment on attachment 445409 [details]
Patch

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

> Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml:170
> +  humanReadableDescription: "Enable 3D transform behavior that specification-compliant but backwards incompatible"

that is specification-compliant
Comment 7 Martin Robinson 2021-12-01 02:08:09 PST
Created attachment 445542 [details]
Patch
Comment 8 EWS 2021-12-01 02:53:48 PST
Committed r286357 (244716@main): <https://commits.webkit.org/244716@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 445542 [details].