Bug 53236

Summary: Web Inspector: [Extensions API] add JSON schema for extensions API
Product: WebKit Reporter: Andrey Kosyakov <caseq>
Component: Web Inspector (Deprecated)Assignee: Andrey Kosyakov <caseq>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, bweinstein, commit-queue, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
patch
none
patch none

Description Andrey Kosyakov 2011-01-27 05:27:32 PST 
This is currently so that docs may be generated for the schema. In the future, the schema should be used for generating client and server API stubs and validating parameters.
Comment 1 Andrey Kosyakov 2011-01-27 05:32:58 PST 
Created attachment 80324 [details]
patch
Comment 2 Andrey Kosyakov 2011-01-27 09:21:56 PST 
Created attachment 80340 [details]
patch

- Cleaned up JSON problems (trailing commas, string concatenation with "+")
- Moved to namespace experimental.webInspector
Comment 3 Pavel Feldman 2011-01-28 01:24:40 PST 
Comment on attachment 80340 [details]
patch

I'd suggest that we place it under WebKit/chromium/src/js.
Comment 4 Andrey Kosyakov 2011-01-28 05:34:10 PST 
(In reply to comment #3)
> (From update of attachment 80340 [details])
> I'd suggest that we place it under WebKit/chromium/src/js.

As discussed offline, we'd like this to be close to ExtensionAPI.js and ExtensionServer.js, as these need to be kept in-sync (manually for now, but ultimately we're looking to generate them from the schema). Also, this is a platform-neutral format that would be handy to generate platform-specific docs from, should other vendors wish to enable WebInspector extensions in their browsers.
Comment 5 WebKit Commit Bot 2011-01-28 08:06:23 PST 
Comment on attachment 80340 [details]
patch

Clearing flags on attachment: 80340

Committed r76945: <http://trac.webkit.org/changeset/76945>
Comment 6 WebKit Commit Bot 2011-01-28 08:06:27 PST 
All reviewed patches have been landed.  Closing bug.