Bug 72186

Summary: Web Inspector: don't remove InspectorBackendStub.js in Release builds
Product: WebKit Reporter: Timothy Hatcher <timothy>
Component: Web Inspector (Deprecated)Assignee: Timothy Hatcher <timothy>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, joepeck, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed Change
timothy: commit-queue-
Proposed Change (round 2) joepeck: review+, timothy: commit-queue-

Description Timothy Hatcher 2011-11-11 14:44:28 PST 
All the *.js files are combined in Release builds. We should keep InspectorBackendStub.js around so it can be used by other front-ends.
Comment 1 Timothy Hatcher 2011-11-11 14:46:25 PST 
Created attachment 114774 [details]
Proposed Change
Comment 2 Timothy Hatcher 2011-11-11 14:59:09 PST 
Created attachment 114779 [details]
Proposed Change (round 2)
Comment 3 Joseph Pecoraro 2011-11-11 15:09:18 PST 
Comment on attachment 114779 [details]
Proposed Change (round 2)

In the first build phase, instead of the `find -not -name InspectorBackendStub`
it would be clearer to me to just:

    rm *.js
    cp "${DERIVED_FILE_DIR}/WebCore/InspectorBackendStub.js" .

But this approach does create some file churn.

r=me, Either way this looks good.
Comment 4 Timothy Hatcher 2011-11-11 15:19:27 PST 
Committed r100035