RESOLVED FIXED 235170
Remove ImplementationLacksVTable IDL attribute
https://bugs.webkit.org/show_bug.cgi?id=235170
Summary Remove ImplementationLacksVTable IDL attribute
Alex Christensen
Reported 2022-01-12 17:41:43 PST
Remove ImplementationLacksVTable IDL attribute
Attachments
Patch (297.24 KB, patch)
2022-01-12 17:43 PST, Alex Christensen
no flags
Patch (277.94 KB, patch)
2022-01-18 09:29 PST, Alex Christensen
no flags
Patch (37.88 KB, patch)
2022-01-18 10:46 PST, Alex Christensen
no flags
Alex Christensen
Comment 1 2022-01-12 17:43:33 PST
Yusuke Suzuki
Comment 2 2022-01-12 18:39:55 PST
Comment on attachment 449016 [details] Patch r=me This is awesome!
Darin Adler
Comment 3 2022-01-16 11:07:54 PST
Wow, don’t want to have to wait for Alex to land this, so tempted to just set commit-queue+ on it!
Alex Christensen
Comment 4 2022-01-18 09:08:45 PST
This will need coordination with the internal builds to land. I think I'll leave it as an allowed attribute, then land that, then remove the internal ignored ImplementationLacksVTable attributes, then remove ImplementationLacksVTable from the list of allowed attributes.
Alex Christensen
Comment 5 2022-01-18 09:29:54 PST
EWS
Comment 6 2022-01-18 10:10:35 PST
Committed r288118 (246132@main): <https://commits.webkit.org/246132@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449397 [details].
Radar WebKit Bug Importer
Comment 7 2022-01-18 10:11:22 PST
Alex Christensen
Comment 8 2022-01-18 10:37:40 PST
Internal commit 0f5ee832212f7d5b552bb117c1428dc65c563f7e
Alex Christensen
Comment 9 2022-01-18 10:46:36 PST
Reopening to attach new patch.
Alex Christensen
Comment 10 2022-01-18 10:46:41 PST
EWS
Comment 11 2022-01-18 11:37:21 PST
Committed r288133 (246140@main): <https://commits.webkit.org/246140@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449403 [details].
Note You need to log in before you can comment on or make changes to this bug.