The "in" keyword for operation arguments has been dropped from the Web IDL spec a long time ago. It brings no additional information and.
Please talk with timo@ and timothy@ before landing the change, as it will affect IDL files for Safari internal builds.
Created attachment 200106 [details] Patch
Comment on attachment 200106 [details] Patch I'd encourage you to land the patch excluding the change in IDLParser.pm. As long as you remove 'in' from WebKit IDL files, it won't break anything. However, removing the 'in' support from IDLParser.pm will break Safari's internal build. You can make the change in a follow-up patch, discussing with timo@ and timothy@.
Created attachment 200111 [details] Patch for landing Reverted change to the IDL parser for now, as suggested by Kentaro.
Created attachment 200112 [details] Patch for landing Tiny changelog fix.
Comment on attachment 200112 [details] Patch for landing Clearing flags on attachment: 200112 Committed r149368: <http://trac.webkit.org/changeset/149368>
All reviewed patches have been landed. Closing bug.
*** Bug 103362 has been marked as a duplicate of this bug. ***
> discussing with timo@ Where does this e-mail come from? That suggestion is incorrect and should be fixed.
(In reply to comment #9) > > discussing with timo@ > > Where does this e-mail come from? That suggestion is incorrect and should be fixed. Alexey, I am not sure what you mean. Do you mean that it is OK to make the parser enforce that the "in" keyword is not used in the IDLs? If you say it will not break Safari builds, I am happy to do this in a follow-up patch.
Sorry that I was unclear. I'm saying that timo@apple.com has not been working on WebKit since 2006. Presumably there is some wiki page somewhere suggesting to ask him, and this page should be fixed.