Bug 235531

Summary: Sort IDL extended attributes alphabetically
Product: WebKit Reporter: Sam Sneddon [:gsnedders] <gsnedders>
Component: WebXRAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, esprehn+autocc, ews-watchlist, kondapallykalyan, msub2official, sam, tyler_w, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=235527
Attachments:
Description Flags
Everywhere where EnabledBySetting comes before Conditional
none
Patch none

Description Sam Sneddon [:gsnedders] 2022-01-24 11:27:20 PST 
Created attachment 449835 [details]
Everywhere where EnabledBySetting comes before Conditional

Per Darin in comment #2 on Bug 235527:

> Comment on attachment 449829 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=449829&action=review
> 
> > Source/WebCore/Modules/webxr/XRFrameRequestCallback.idl:31
> > +    EnabledBySetting=WebXREnabled,
> >      Conditional=WEBXR
> 
> Normally we like to keep these alphabetical.

The existing IDL should be fixed to adhere to this; this is almost entirely in WebXR & related testing IDL.
Comment 1 Radar WebKit Bug Importer 2022-01-31 11:28:19 PST 
<rdar://problem/88284366>
Comment 2 Daniel Adams 2022-03-27 01:07:06 PDT 
Created attachment 455864 [details]
Patch
Comment 3 Sam Weinig 2022-03-27 08:36:28 PDT 
Comment on attachment 455864 [details]
Patch

We should probably enforce this in the IDL parser for good measure at some point.
Comment 4 EWS 2022-03-27 19:36:24 PDT 
Committed r291954 (248921@main): <https://commits.webkit.org/248921@main>

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