RESOLVED FIXED28813
REGRESSION: Debugger no longer lists all script sources
https://bugs.webkit.org/show_bug.cgi?id=28813
Summary REGRESSION: Debugger no longer lists all script sources
Oliver Hunt
Reported 2009-08-28 11:12:26 PDT
The debugger no longer lists the full set of scripts that are available. To Reproduce: 1. Go to http://testsuites.opera.com/JSON/correctness/004.html 2. Enable debugging 3. Look at the JS code drop down in the debugger window In r47839 there are only 3 entries: (program) 004.js common.js In 4.0.3 there is also 004.html (as this has script as well)
Attachments
Pavel Feldman
Comment 1 2009-09-01 14:34:13 PDT
In addition to the problem reported, scripts do not have content available. The reason is that scripts are being reported as parsed prior to corresponding resources being reported as loaded. As a result, wrong viewer is being instantiated for the resource (resource type is "Other" during the parsedScript invocation). [SubresourceLoader:137] m_client->didReceiveResponse(this, r); leads to InspectorController::didParseSource call, whereas [SubresourceLoader:142] ResourceLoader::didReceiveResponse(r); leads to the InspectorController::didReceiveResponse notification. Frontend should either expect this order of events or notifications order should be fixed. Timothy, what is your view on this?
Timothy Hatcher
Comment 2 2009-09-01 15:55:22 PDT
Yes, that is a correct diagnosis. This has been an issue from the beginning in some cases. But maybe it happens more now. I think we can fix it either way. Might be easier to fix the order the InspectorController is notified.
Jon Evans
Comment 3 2009-11-04 05:31:21 PST
I think I'm seeing this bug. Scripts appear in the drop-down but don't display when you select them. Really frustrating if you're following code paths that span different js files. Sometimes reloading the page helps, but not always.
Timothy Hatcher
Comment 4 2009-11-08 08:27:17 PST
*** Bug 31230 has been marked as a duplicate of this bug. ***
Alexey Proskuryakov
Comment 5 2009-11-08 11:42:26 PST
See also: bug 30506.
Note You need to log in before you can comment on or make changes to this bug.