As part of efforts to finally add an 'undefined' type to WebIDL (so methods/etc don't have to generalize their type to 'any'), it was decided to actually add 'undefined' as the type name, and so the previous function return signature 'void' is being removed for redundancy. See https://github.com/heycam/webidl/pull/906 for details on the change. This change will be merged shortly; this bug is a heads-up to adjust your IDL parser (see the PR for the changes to the grammar; it's quite small). You may wish to have a transitional period where you recognize 'void' and 'undefined' as synonyms, as specifications are moved over to the new syntax.
*** Bug 215652 has been marked as a duplicate of this bug. ***
<rdar://problem/67566201>
Created attachment 407481 [details] Patch
Comment on attachment 407481 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407481&action=review > Source/WebCore/ChangeLog:10 > + Also updates the parser to better match current WebIDl grammar, removing the special WebIDL > Source/WebCore/bindings/scripts/test/JS/JSTestIterable.cpp:200 > - using KeyType = void; > + using KeyType = undefined; Looks like this change in expected results is wrong. > Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp:388 > - using KeyType = void; > + using KeyType = undefined; Ditto.
(In reply to Darin Adler from comment #4) > Comment on attachment 407481 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=407481&action=review > > > Source/WebCore/ChangeLog:10 > > + Also updates the parser to better match current WebIDl grammar, removing the special > > WebIDL Fixing. > > > Source/WebCore/bindings/scripts/test/JS/JSTestIterable.cpp:200 > > - using KeyType = void; > > + using KeyType = undefined; > > Looks like this change in expected results is wrong. > > > Source/WebCore/bindings/scripts/test/JS/JSTestNode.cpp:388 > > - using KeyType = void; > > + using KeyType = undefined; > > Ditto. Oh, yeah, didn't run run-bindings-test --reset-results after I fixed that. Fixing.
Created attachment 407484 [details] Patch
Tools/Scripts/svn-apply failed to apply attachment 407484 [details] to trunk. Please resolve the conflicts and upload a new patch.
Created attachment 407507 [details] Patch
Committed r266311: <https://trac.webkit.org/changeset/266311> All reviewed patches have been landed. Closing bug and clearing flags on attachment 407507 [details].