Bug 63687 - Web Inspector: remove ExtensionAPISchema.json
Summary: Web Inspector: remove ExtensionAPISchema.json
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Andrey Kosyakov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-29 23:57 PDT by Andrey Kosyakov
Modified: 2011-06-30 07:03 PDT (History)
11 users (show)

See Also:


Attachments
patch (15.62 KB, patch)
2011-06-29 23:59 PDT, Andrey Kosyakov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Kosyakov 2011-06-29 23:57:54 PDT
We don't use the extension API schema for anything in WebCore now, so it shouldn't be there.
Comment 1 Andrey Kosyakov 2011-06-29 23:59:51 PDT
Created attachment 99247 [details]
patch
Comment 2 Patrick Mueller 2011-06-30 04:17:59 PDT
I assume that the extension function isn't going away, just the json schema describing it?  

Previously some code was previously using the schema for code-gen or validating AP, but now it's not?
Comment 3 Andrey Kosyakov 2011-06-30 04:49:49 PDT
(In reply to comment #2)
> I assume that the extension function isn't going away, just the json schema describing it?  

Exactly.

> Previously some code was previously using the schema for code-gen or validating AP, but now it's not?

While both validation and generating code are something we consider in the future, we don't do this yet (and never did). The only purpose of this schema was generating docs, and the doc generating code was never in the WebCore land, so we decided to remove the schema for now. It may come back if we go for validation and/or code generation.
Comment 4 WebKit Review Bot 2011-06-30 07:03:42 PDT
Comment on attachment 99247 [details]
patch

Clearing flags on attachment: 99247

Committed r90116: <http://trac.webkit.org/changeset/90116>
Comment 5 WebKit Review Bot 2011-06-30 07:03:47 PDT
All reviewed patches have been landed.  Closing bug.