[Webkit-unassigned] [Bug 108589] REGRESSION (r137607): Loading of archives as substitute data is broken

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 1 15:31:17 PST 2013


--- Comment #4 from Nate Chapin <japhet at chromium.org>  2013-02-01 15:33:19 PST ---
(In reply to comment #3)
> Do you think that other callers of this function would be OK with that? An archive is not really truly the main resource, even though it's handled like that at the first stage of loading.

Another option, it appears that we can safely exit early and get the right result in DocumentLoader::maybeCreateArchive() if mainResourceData() returns null. I don't know whether that has side effects, though.

I'm trying to decide whether there are other situations where this underlying issue (Non-empty SubstituteData loads leave DocumentLoader::mainResourceData() empty) could be a problem. My guess is Yes, but I don't have definitive proof.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list