Bug 42075

Summary: update scriptIDToContent before calling frontend->parsedScriptSource
Product: WebKit Reporter: Jamey Hicks <jamey.hicks>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, joepeck, pfeldman
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch pfeldman: review+

Jamey Hicks
Reported 2010-07-12 06:55:29 PDT
update scriptIDToContent before calling frontend->parsedScriptSource
Attachments
Patch (2.80 KB, patch)
2010-07-12 07:41 PDT, Jamey Hicks 		pfeldman: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jamey Hicks
Comment 1 2010-07-12 07:41:14 PDT
Created attachment 61229 [details] Patch
Joseph Pecoraro
Comment 2 2010-07-18 18:14:31 PDT
Patch looks okay, but Pavel would probably know best. @Jamey, you can use http://webkit.org/new-inspector-bug when creating inspector bugs to immediately get some people CC'd.
Pavel Feldman
Comment 3 2010-07-19 00:14:56 PDT
Comment on attachment 61229 [details] Patch Patch looks fine, however it should not be necessary. Backend -> frontend interaction is asynchronous, so by the time frontend queries for the content, it should already bound. Is there a user scenario suffering from the issue? (Putting r+ for nice includes cleanup anyways).
Adam Barth
Comment 4 2010-08-10 22:40:04 PDT
This appears to have already landed.
Note You need to log in before you can comment on or make changes to this bug.