[webkit-reviews] review canceled: [Bug 112811] REGRESSION (r146239): Reproducible crash in WebCore::DocumentLoader::responseReceived : [Attachment 194370] patch

[bugzilla-daemon at webkit.org]

Nate Chapin <japhet at chromium.org> has canceled Nate Chapin
<japhet at chromium.org>'s request for review:
Bug 112811: REGRESSION (r146239): Reproducible crash in
WebCore::DocumentLoader::responseReceived
https://bugs.webkit.org/show_bug.cgi?id=112811

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

------- Additional Comments from Nate Chapin <japhet at chromium.org>
Ok, I *think* this does the job.  In a normal load, we set a main resource's
identifier on the CachedRawResource when we receive a response, but we don't do
so in the revalidation case.  Since a resource can't have multiple
revalidations at once and can't be revalidated if it's still loading, it should
be safe to copy the identifier to the revalidated resource, even though that
overwrites its old identifier.