Bug 36658

Summary: Web Inspector: contents of worker script not shown on the resource tab
Product: WebKit Reporter: Andrey Kosyakov <caseq>
Component: Web Inspector (Deprecated)Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bweinstein, commit-queue, dglazkov, joepeck, keishi, pfeldman, pmuellr, rik, timothy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab
none
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab none

Description Andrey Kosyakov 2010-03-26 05:31:56 PDT 
1. Load a page with workers
2. Workers debugging should be set to off (i.e. not using fake workers)

Note worker scripts are shown in resource panel, but content tab is empty.
Comment 1 Andrey Kosyakov 2010-03-26 05:40:50 PDT 
Created attachment 51733 [details]
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab
Comment 2 WebKit Review Bot 2010-03-26 05:43:20 PDT 
Attachment 51733 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/1387017
Comment 3 Andrey Kosyakov 2010-03-26 06:36:49 PDT 
Created attachment 51736 [details]
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab
Comment 4 Pavel Feldman 2010-03-26 06:52:38 PDT 
Comment on attachment 51736 [details]
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab

I noticed that there is a number of methods on ScriptExecutionContext that are there with no good reason. All they do is delegate to InspectorController instance that is now exposed on the context itself. Since you exposed it in the context, could you remove these methods now?
Comment 5 WebKit Commit Bot 2010-03-26 09:34:21 PDT 
Comment on attachment 51736 [details]
Send worker resource content to inspector controller to enable display of worker scripts content in inspector's resource tab

Clearing flags on attachment: 51736

Committed r56630: <http://trac.webkit.org/changeset/56630>
Comment 6 WebKit Commit Bot 2010-03-26 09:34:26 PDT 
All reviewed patches have been landed.  Closing bug.