WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
98980
meta: WebKit IDL syntax should conform to the Web IDL syntax
https://bugs.webkit.org/show_bug.cgi?id=98980
Summary
meta: WebKit IDL syntax should conform to the Web IDL syntax
Kentaro Hara
Reported
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/
).
Attachments
Add attachment
proposed patch, testcase, etc.
Kentaro Hara
Comment 1
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?
Adam Barth
Comment 2
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?
Kentaro Hara
Comment 3
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:-)
Dirk Schulze
Comment 4
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).
Adam Barth
Comment 5
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.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug