RESOLVED FIXED 51243
Web Inspector: can't set breakpoints if resource content is loaded too fast 
https://bugs.webkit.org/show_bug.cgi?id=51243
Summary Web Inspector: can't set breakpoints if resource content is loaded too fast
Pavel Podivilov
Reported 2010-12-17 04:20:25 PST
Currently if resource content is loaded too fast, SourceFrame doesn't get all the scripts, so SourceFrame should listen for parsed script events itself.
Attachments
Patch. (4.31 KB, patch)
2010-12-17 04:21 PST, Pavel Podivilov 		pfeldman: review-
DetailsFormatted DiffDiff
Patch. (1.99 KB, patch)
2010-12-17 05:09 PST, Pavel Podivilov 		pfeldman: review-
DetailsFormatted DiffDiff
Patch. (1.97 KB, patch)
2010-12-17 06:45 PST, Pavel Podivilov 		pfeldman: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Pavel Podivilov
Comment 1 2010-12-17 04:21:22 PST
Created attachment 76865 [details] Patch.
Pavel Feldman
Comment 2 2010-12-17 04:32:48 PST
Comment on attachment 76865 [details] Patch. I don't think SourceFrame should listen to parsed script events.
Pavel Podivilov
Comment 3 2010-12-17 05:09:34 PST
Created attachment 76870 [details] Patch. Add scripts parsed after resource load to SourceFrame.
Pavel Feldman
Comment 4 2010-12-17 06:19:13 PST
Comment on attachment 76870 [details] Patch. View in context: https://bugs.webkit.org/attachment.cgi?id=76870&action=review > WebCore/inspector/front-end/ScriptsPanel.js:259 > + var view = WebInspector.ResourceManager.resourceViewForResource(resource); You should use WebInspector.ResourceManager.existingResourceViewForResource not to create it unless necessary.
Pavel Podivilov
Comment 5 2010-12-17 06:45:14 PST
Created attachment 76875 [details] Patch.
Pavel Podivilov
Comment 6 2010-12-17 06:48:13 PST
Committed r74261: <http://trac.webkit.org/changeset/74261>
Note You need to log in before you can comment on or make changes to this bug.