Bug 98980 - meta: WebKit IDL syntax should conform to the Web IDL syntax
Summary: meta: WebKit IDL syntax should conform to the Web IDL syntax
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Takashi Sakamoto
URL:
Keywords:
Depends on: 99160 99007 99012 103362
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-10 19:25 PDT by Kentaro Hara
Modified: 2017-01-18 23:39 PST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kentaro Hara 2012-10-10 19:25:58 PDT
A new sane IDL parser landed at r129723 (https://bugs.webkit.org/show_bug.cgi?id=26398), and now it's easy to modify WebKit IDL syntax. We should remove legacy WebKit IDL syntax from IDL files and make them conform to the Web IDL spec (http://www.w3.org/TR/WebIDL/).
Comment 1 Kentaro Hara 2012-10-10 19:46:07 PDT
tasak: To clarify what we should do, would you roughly list up the legacy WebKit IDL syntaxes you are going to remove?
Comment 2 Adam Barth 2012-10-10 19:53:57 PDT
Maybe it would make sense to change one IDL file at a time, perhaps starting with a simple one?
Comment 3 Kentaro Hara 2012-11-19 18:10:18 PST
tasak: Our objective is to remove xxxOld() methods from the IDL parser. I think we've removed almost all (or all?) old syntaxes from WebKit IDL files, but the Safari's internal build is still using a couple of old syntaxes. We need to ask Apple guys to remove the old syntaxes before removing xxxOld() from the IDL parser. Would you list up old syntaxes that should be removed from their IDL files? (If you are busy, I'm happy to write patches. But I'm not fully familiar with old syntaxes like you:-)
Comment 4 Dirk Schulze 2012-11-27 07:46:11 PST
Do we already support "implements"? This would be necessary to move SVG to WebIDL (which has multiple inheritance at the moment).
Comment 5 Adam Barth 2012-11-27 09:53:15 PST
> Do we already support "implements"?

Our current implementation of implements has a slightly different syntax, but we should be able to support "implements" without too much trouble.