[webkit-reviews] review requested: [Bug 49693] REGRESSION(71884): Crash when installing extension that references safari-extension URL : [Attachment 74281] new patch + test

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 18 13:11:43 PST 2010


Nate Chapin <japhet at chromium.org> has asked  for review:
Bug 49693: REGRESSION(71884): Crash when installing extension that references
safari-extension URL
https://bugs.webkit.org/show_bug.cgi?id=49693

Attachment 74281: new patch + test
https://bugs.webkit.org/attachment.cgi?id=74281&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
This version moves both removing from the queue and adding to the set of loads
in progress.  By doing them both before, we can trust
ResourceLoadScheduler::remove() to do the right thing without any special logic
in servePendingRequests(), even when called syncrhonously.


More information about the webkit-reviews mailing list