[webkit-reviews] review requested: [Bug 61225] Cache XHRs (and other ThreadableLoaderClients) : [Attachment 112252] Address issues from revert

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Oct 24 14:46:38 PDT 2011


Nate Chapin <japhet at chromium.org> has asked  for review:
Bug 61225: Cache XHRs (and other ThreadableLoaderClients)
https://bugs.webkit.org/show_bug.cgi?id=61225

Attachment 112252: Address issues from revert
https://bugs.webkit.org/attachment.cgi?id=112252&action=review

------- Additional Comments from Nate Chapin <japhet at chromium.org>
Diff from last version:

CachedRawResources default to ResourceLoadPriorityMedium.  This matches the
existing behavior for DocumentThreadableLoaders.

CachedRawResource::data() protects the CachedRawResource.  The crash described
in https://bugs.webkit.org/show_bug.cgi?id=70388 was caused by the
CachedRawResource being deleted prematurely by the dataReceived() call.


More information about the webkit-reviews mailing list