Get main frame and subframe AppCache loads to work.

content/browser/appcache/appcache_request_handler.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/appcache/appcache_request_handler.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "content/browser/appcache/appcache.h"
 #include "content/browser/appcache/appcache_backend_impl.h"
+#include "content/browser/appcache/appcache_host.h"
+#include "content/browser/appcache/appcache_navigation_handle_core.h"
 #include "content/browser/appcache/appcache_policy.h"
 #include "content/browser/appcache/appcache_request.h"
+#include "content/browser/appcache/appcache_url_loader_factory.h"
+#include "content/browser/appcache/appcache_url_loader_job.h"
+#include "content/browser/appcache/appcache_url_loader_request.h"
 #include "content/browser/appcache/appcache_url_request_job.h"
 #include "content/browser/service_worker/service_worker_request_handler.h"
+#include "content/browser/url_loader_factory_getter.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_job.h"
 
 namespace content {
 
 AppCacheRequestHandler::AppCacheRequestHandler(
     AppCacheHost* host,
     ResourceType resource_type,
     bool should_reset_appcache,
     std::unique_ptr<AppCacheRequest> request)
@@ skipping 190 matching lines @@
 
 void AppCacheRequestHandler::MaybeCompleteCrossSiteTransferInOldProcess(
     int old_process_id) {
   if (!host_ || !host_for_cross_site_transfer_.get() ||
       old_process_id != old_process_id_) {
     return;
   }
   CompleteCrossSiteTransfer(old_process_id_, old_host_id_);
 }
 
+// static
+std::unique_ptr<AppCacheRequestHandler>
+AppCacheRequestHandler::InitializeForNavigationNetworkService(
+    const ResourceRequest& request,
+    AppCacheNavigationHandleCore* appcache_handle_core,
+    URLLoaderFactoryGetter* url_loader_factory_getter) {
+  std::unique_ptr<AppCacheRequestHandler> handler =

[jam, 2017/06/07 01:42:17]

I thought an earlier patchset didn't create an appcache handler if that origin
didn't have any appcaches associated, or am I misremembering?

[michaeln, 2017/06/07 02:53:45]

to support redirects to appcached pages, we can't do that check at this point

[ananta, 2017/06/07 20:57:16]

You are correct there. However we decided to go with the handler based approach
for consistency with ServiceWorker.

+      appcache_handle_core->host()->CreateRequestHandler(

[jam, 2017/06/07 01:42:17]

can't this return null?

[michaeln, 2017/06/07 02:53:45]

yes, it can

[ananta, 2017/06/07 20:57:16]

yes

+          AppCacheURLLoaderRequest::Create(request), request.resource_type,
+          request.should_reset_appcache);
+  handler->url_loader_factory_getter_ = url_loader_factory_getter;
+  return handler;
+}
+
 void AppCacheRequestHandler::OnDestructionImminent(AppCacheHost* host) {
   storage()->CancelDelegateCallbacks(this);
   host_ = NULL;  // no need to RemoveObserver, the host is being deleted
 
   // Since the host is being deleted, we don't have to complete any job
   // that is current running. It's destined for the bit bucket anyway.
   if (job_.get()) {
     job_->Kill();
     job_.reset();
   }
@@ skipping 21 matching lines @@
   DCHECK(host_ && job_.get() && job_->IsWaiting());
   DCHECK(entry.has_response_id());
 
   // Cache information about the response, for use by GetExtraResponseInfo.
   cache_id_ = cache_id;
   manifest_url_ = manifest_url;
 
   if (IsResourceTypeFrame(resource_type_) && !namespace_entry_url.is_empty())
     host_->NotifyMainResourceIsNamespaceEntry(namespace_entry_url);
 
+  if (navigation_request_job_.get()) {
+    appcache_factory_ = AppCacheURLLoaderFactory::CreateURLLoaderFactory(
+        url_loader_factory_getter_.get(),
+        navigation_request_job_.release());
+    std::move(loader_factory_callback_).Run(appcache_factory_.get());

[michaeln, 2017/06/07 02:53:45]

Do we still need to retain the appcache_factory_ instance beyond this point? It
looks like kinuko's CL to get rid of the factory is good to go, this should go
away after rebasing.

[ananta, 2017/06/07 20:57:16]

Will look into that. I am not sure her patch landed which gets rid of the
factory. For now we need this.

[michaeln, 2017/06/08 02:54:00]

This logic also could be hoisted into the AppCacheUrlLoaderJob class, so the
callback can be deferred further, until the response info load completes.

[ananta, 2017/06/08 05:58:39]

Done.

+  }
   job_->DeliverAppCachedResponse(manifest_url, cache_id, entry, is_fallback);
 }
 
 void AppCacheRequestHandler::DeliverErrorResponse() {
   DCHECK(job_.get() && job_->IsWaiting());
   DCHECK_EQ(kAppCacheNoCacheId, cache_id_);
   DCHECK(manifest_url_.is_empty());
   job_->DeliverErrorResponse();
 }
 
 void AppCacheRequestHandler::DeliverNetworkResponse() {
   DCHECK(job_.get() && job_->IsWaiting());
   DCHECK_EQ(kAppCacheNoCacheId, cache_id_);
   DCHECK(manifest_url_.is_empty());
+
+  // In network service land, if we are processing a navigation request, we
+  // need to inform the loader callback that we are not going to handle this
+  // request.
+  if (navigation_request_job_.get()) {

[michaeln, 2017/06/08 02:54:00]

This bit of logic to call the loadercallback could go inside
AppCacheUrlLoaderJob::DeliverNetworkResponse(), that would keep this class more
agnostic about when loading system was being used.

[ananta, 2017/06/08 05:58:39]

Done.

+    std::move(loader_factory_callback_).Run(nullptr);
+    navigation_request_job_.reset(nullptr);
+    return;
+  }
+
   job_->DeliverNetworkResponse();
 }
 
 void AppCacheRequestHandler::OnPrepareToRestart() {
   DCHECK(job_->IsDeliveringNetworkResponse() || job_->IsCacheEntryNotFound());
 
   // Any information about the source of the response is no longer relevant.
   cache_id_ = kAppCacheNoCacheId;
   manifest_url_ = GURL();
 
@@ skipping 18 matching lines @@
 // Main-resource handling ----------------------------------------------
 
 std::unique_ptr<AppCacheJob> AppCacheRequestHandler::MaybeLoadMainResource(
     net::NetworkDelegate* network_delegate) {
   DCHECK(!job_.get());
   DCHECK(host_);
 
   // If a page falls into the scope of a ServiceWorker, any matching AppCaches
   // should be ignored. This depends on the ServiceWorker handler being invoked
   // prior to the AppCache handler.
-  if (ServiceWorkerRequestHandler::IsControlledByServiceWorker(
+  // TODO(ananta)
+  // We need to handle this for AppCache requests initiated for the network
+  // service
+  if (request_->GetURLRequest() &&
+      ServiceWorkerRequestHandler::IsControlledByServiceWorker(
           request_->GetURLRequest())) {
     host_->enable_cache_selection(false);
     return nullptr;
   }
 
+  if (storage()->IsInitialized() &&
+      service_->storage()->usage_map()->find(request_->GetURL().GetOrigin()) ==

[michaeln, 2017/06/07 02:53:45]

@jam, this is where the origin test goes

+          service_->storage()->usage_map()->end()) {
+    return nullptr;
+  }
+
   host_->enable_cache_selection(true);
 
   const AppCacheHost* spawning_host =
       (resource_type_ == RESOURCE_TYPE_SHARED_WORKER) ?
       host_ : host_->GetSpawningHost();
   GURL preferred_manifest_url = spawning_host ?
       spawning_host->preferred_manifest_url() : GURL();
 
   // We may have to wait for our storage query to complete, but
   // this query can also complete syncrhonously.
@@ skipping 161 matching lines @@
 
   if (!host_->associated_cache() ||
       !host_->associated_cache()->is_complete()) {
     DeliverNetworkResponse();
     return;
   }
 
   ContinueMaybeLoadSubResource();
 }
 
+void AppCacheRequestHandler::MaybeCreateLoaderFactory(
+    const ResourceRequest& resource_request,
+    ResourceContext* resource_context,
+    LoaderFactoryCallback callback) {
+  navigation_request_job_ = MaybeLoadMainResource(nullptr);
+  if (!navigation_request_job_.get()) {
+    std::move(callback).Run(nullptr);
+    return;
+  }
+  navigation_request_job_->AsURLLoaderJob()->SetURLLoaderFactoryGetter(
+      url_loader_factory_getter_.get());
+  loader_factory_callback_ = std::move(callback);

[michaeln, 2017/06/08 02:54:00]

This datamember could go inside the AppCacheUrlLoaderJob, giving it the
responsibility to call it appropriately with nullptr or itself.

[ananta, 2017/06/08 05:58:39]

Done.

+}
+
 }  // namespace content