RESOLVED FIXED 230584
[WebIDL] Align property order of DOM constructors with ECMA-262 counterparts
https://bugs.webkit.org/show_bug.cgi?id=230584
Summary [WebIDL] Align property order of DOM constructors with ECMA-262 counterparts
Attachments
Patch (145.61 KB, patch)
2021-09-25 13:09 PDT, Alexey Shvayka
no flags
Patch (145.02 KB, patch)
2021-09-25 21:37 PDT, Alexey Shvayka
no flags
Patch (145.72 KB, patch)
2021-09-26 00:25 PDT, Alexey Shvayka
no flags
Patch (132.05 KB, patch)
2021-09-27 17:47 PDT, Alexey Shvayka
no flags
Patch (132.08 KB, patch)
2021-09-28 05:59 PDT, Alexey Shvayka
no flags
Alexey Shvayka
Comment 1 2021-09-25 13:09:42 PDT
EWS Watchlist
Comment 2 2021-09-25 13:10:55 PDT
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Alexey Shvayka
Comment 3 2021-09-25 21:37:27 PDT
Created attachment 439283 [details] Patch Bring back [LegacyFactoryFunctionEnabledBySetting] attribute and attempt to fix WPE build by exporting constructor's finishCreation() methods.
Alexey Shvayka
Comment 4 2021-09-26 00:25:43 PDT
Created attachment 439291 [details] Patch Attempt to fix WPE build by populating TestWebCore_LIBRARIES.
Radar WebKit Bug Importer
Comment 5 2021-09-27 10:18:29 PDT
Alexey Shvayka
Comment 6 2021-09-27 17:47:01 PDT
Created attachment 439420 [details] Patch Abolish calling into InternalFunction::finishCreation(VM&, unsigned length, const String& name) and instead merely reorder properties.
Alexey Shvayka
Comment 7 2021-09-28 05:59:15 PDT
Created attachment 439464 [details] Patch Fix: don't overwrite "length" that was defined via [LegacyFactoryFunctionEnabledBySetting] attribute
EWS
Comment 8 2021-09-29 10:17:12 PDT
Committed r283233 (242275@main): <https://commits.webkit.org/242275@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 439464 [details].
Alexey Shvayka
Comment 9 2021-09-30 09:10:03 PDT
Thank you for review, Alex!
Note You need to log in before you can comment on or make changes to this bug.