[webkit-reviews] review granted: [Bug 42075] update scriptIDToContent before calling frontend->parsedScriptSource : [Attachment 61229] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jul 19 00:14:55 PDT 2010


Pavel Feldman <pfeldman at chromium.org> has granted Jamey Hicks
<jamey.hicks at nokia.com>'s request for review:
Bug 42075: update scriptIDToContent before calling frontend->parsedScriptSource
https://bugs.webkit.org/show_bug.cgi?id=42075

Attachment 61229: Patch
https://bugs.webkit.org/attachment.cgi?id=61229&action=review

------- Additional Comments from Pavel Feldman <pfeldman at chromium.org>
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).


More information about the webkit-reviews mailing list