Bug 120242

Summary: Web Inspector: We should regenerate InspectorBackendCommands.js for Legacy Inspector.json versions
Product: WebKit Reporter: Joseph Pecoraro <joepeck>
Component: Web InspectorAssignee: Joseph Pecoraro <joepeck>
Status: RESOLVED FIXED    
Severity: Normal CC: burg, graouts, joepeck, timothy, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
[PATCH] Proposed Fix timothy: review+

Description Joseph Pecoraro 2013-08-23 17:52:02 PDT
Older Inspector.json files could benefit from the enum values, which is just new sugar generated from the Inspector.json that existed. We should regenerate the InspectorBackendCommands.js for older, fixed versions. And we should include the older Inspector.json files so that we can easily compare against this. For starters, this should happen with iOS 6's Legacy/6.0/InspectorBackendCommands.js.
Comment 1 Radar WebKit Bug Importer 2013-08-23 17:52:49 PDT
<rdar://problem/14825561>
Comment 2 Joseph Pecoraro 2013-08-23 17:58:05 PDT
Created attachment 209528 [details]
[PATCH] Proposed Fix

- includes the Inspector.json for iOS 6.0.
- includes a script for regenerating the Legacy/6.0/InspectorBackendCommands.js.
- some other tweaks to improve output
Comment 3 Joseph Pecoraro 2013-08-23 18:08:21 PDT
FWIW, iOS 6.1 was identical to iOS 6.0.
Comment 4 Timothy Hatcher 2013-08-24 08:10:26 PDT
Comment on attachment 209528 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=209528&action=review

> Source/WebInspectorUI/ChangeLog:9
> +        - Provide a helper script to regenerate the backend commands file for trunk and Versions.

We should run this every build eventually. Maybe we should disable the JS command generation in WebCore.
Comment 5 Joseph Pecoraro 2013-08-26 13:38:17 PDT
(In reply to comment #4)
> (From update of attachment 209528 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=209528&action=review
> 
> > Source/WebInspectorUI/ChangeLog:9
> > +        - Provide a helper script to regenerate the backend commands file for trunk and Versions.
> 
> We should run this every build eventually. Maybe we should disable the JS command generation in WebCore.

Good idea. I'd love to do that once we've had all clients switch to the new inspector frontend! It is pretty fast, and if we could make a few changes to only generate the JS file for older inputs.
Comment 6 Joseph Pecoraro 2013-08-26 16:39:01 PDT
Committed <http://trac.webkit.org/changeset/154651>.

Sorry for landing with "OOPs" in the commit message. It did have a reviewer in the ChangeLogs!
Comment 7 Brian Burg 2013-09-30 13:27:50 PDT
*** Bug 117843 has been marked as a duplicate of this bug. ***