Bug 66163
| Summary: | Loading resources in reparented iframes are cancelled when original window closes. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Dmitry Titov <dimich> |
| Component: | WebCore Misc. | Assignee: | Dmitry Titov <dimich> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, aroben, beidson, bweinstein, fishd, jennb, sam |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | 66165, 66167 | ||
| Bug Blocks: | |||
Dmitry Titov
This is an umbrella bug for patches required to fix http://crbug.com/55200
We need to have a mechanism to tell the embedder that a resource which is still loading is now owned by another WebFrame. The patch for the bug 44713 has introduced "transferLoadingResourceToNewPage" notification on FrameLoaderClient which is very close in semantics. Unfortunately, it doesn't come with enough parameters to let embedders like Chromium to 'redirect' the loading notifications to the new top-level WebFrame.
The idea of the fix is to pass ResourceLoader of the loading resource into transferLoadingResourceToNewPage. This will make it possible to get to the platform-specific ResourceHandleInternal on the side of the embedder, so the embedder-specific info may be pulled out of it and transferred to the WebFrame which is the new owner of loading resources.
This obsoletes bug 65890 which was the previous attempt at the fix.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Dmitry Titov
WebKit now provides a FrameLoaderCient callback that has sufficient info for the embedder to find the pending resource load and update it if necessary. Chromium's embedder side patch was landed here: http://src.chromium.org/viewvc/chrome?view=rev&revision=97048