Bug 19053 - Source code not visible when script is paused before onload fires
Summary: Source code not visible when script is paused before onload fires
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: Timothy Hatcher
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2008-05-14 12:31 PDT by Anthony Ricaud
Modified: 2008-10-04 21:13 PDT (History)
1 user (show)

See Also:

Attachments
Testcase (536 bytes, text/html)
2008-05-14 12:33 PDT, Anthony Ricaud 		no flags Details
Proposed patch (10.40 KB, patch)
2008-10-04 21:03 PDT, Timothy Hatcher 		mrowe: review+
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Anthony Ricaud 2008-05-14 12:31:56 PDT 
Steps to reproduce:
1. Open the testcase
2. Open and attach the Inspector's debugger
3. Put a breakpoint on the "var p =" line.
4. Refresh the page

The script is now paused but the source code is not visible in the Scripts tab.
Comment 1 Anthony Ricaud 2008-05-14 12:33:18 PDT 
Created attachment 21130 [details]
Testcase
Comment 2 Timothy Hatcher 2008-05-15 12:22:53 PDT 
<rdar://problem/5933408>
Comment 3 Anthony Ricaud 2008-05-29 08:14:33 PDT 
Apparently, this is happening for scripts running before the end of onload event.

From IRC :
aroben: Rik|work: I bet your script is running before onload fires
Rik|work: aroben: source code is not printed even in a onload function
aroben: Rik|work: yeah, nothing seems to work until after onload
Comment 4 Timothy Hatcher 2008-05-29 08:17:56 PDT 
We don't show source because the resource hasn't finished loading untill after onload fires. We need a way to show source before the load finishes.
Comment 5 Timothy Hatcher 2008-10-04 21:03:17 PDT 
Created attachment 24098 [details]
Proposed patch
Comment 6 Mark Rowe (bdash) 2008-10-04 21:06:32 PDT 
Comment on attachment 24098 [details]
Proposed patch

r=me
Comment 7 Timothy Hatcher 2008-10-04 21:13:33 PDT 
Landed in r37313.