diff options
| author | michaeln@chromium.org <michaeln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-04 00:34:19 +0000 |
|---|---|---|
| committer | michaeln@chromium.org <michaeln@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-04 00:34:19 +0000 |
| commit | 5fcb289ebad2277b30a19927098f8032dadb5dd1 (patch) | |
| tree | 1bfcf3d665d2e10b1441cbeb0820609cfb2ef4bb /content/browser/loader/cross_site_resource_handler.cc | |
| parent | 86e75f009ef286aed6f7a534f49db57bc08239a7 (diff) | |
| download | chromium_src-5fcb289ebad2277b30a19927098f8032dadb5dd1.zip chromium_src-5fcb289ebad2277b30a19927098f8032dadb5dd1.tar.gz chromium_src-5fcb289ebad2277b30a19927098f8032dadb5dd1.tar.bz2 | |
Transfer appcache state during cross site navigations too.
The AppCacheHost is an object that persist across the lifetime of a document. It's created prior to the document and prior to the initiation of the resource load for the documents main resource. Today, when a cross-site xfer occurs the host is deleted and a new host is created that knows nothing of whats going on... so not surprisingly things are broken.
Either state of the old host that gets deleted needs to be saved and restored into a new host that gets created... or the old host should not get deleted and instead get transferred along as well.
I chose the latter because there's some state that can't be saved/restored so easily and moving the whole object is less tedious.
BUG=331270
Review URL: https://codereview.chromium.org/137883005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248631 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content/browser/loader/cross_site_resource_handler.cc')
| -rw-r--r-- | content/browser/loader/cross_site_resource_handler.cc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/content/browser/loader/cross_site_resource_handler.cc b/content/browser/loader/cross_site_resource_handler.cc index 8fb1c41..5bed767 100644 --- a/content/browser/loader/cross_site_resource_handler.cc +++ b/content/browser/loader/cross_site_resource_handler.cc @@ -24,6 +24,7 @@ #include "content/public/common/url_constants.h" #include "net/http/http_response_headers.h" #include "net/url_request/url_request.h" +#include "webkit/browser/appcache/appcache_interceptor.h" namespace content { @@ -289,6 +290,8 @@ void CrossSiteResourceHandler::StartCrossSiteTransition( referrer = Referrer(GURL(request()->referrer()), info->GetReferrerPolicy()); frame_id = info->GetFrameID(); + appcache::AppCacheInterceptor::PrepareForCrossSiteTransfer( + request(), global_id.child_id); ResourceDispatcherHostImpl::Get()->MarkAsTransferredNavigation(global_id); } BrowserThread::PostTask( |