Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(204)

Unified Diff: content/browser/service_worker/service_worker_context_request_handler_unittest.cc

Issue 2575523002: Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch (Closed)
Patch Set: drop prefetch url request Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/service_worker/service_worker_context_request_handler_unittest.cc
diff --git a/content/browser/service_worker/service_worker_context_request_handler_unittest.cc b/content/browser/service_worker/service_worker_context_request_handler_unittest.cc
index 5c6a1d7a1e1d1f0d8310b5d2b6607336a285b06b..68435616f5b201ad69b5a8b5f0ae20992e5ba7a6 100644
--- a/content/browser/service_worker/service_worker_context_request_handler_unittest.cc
+++ b/content/browser/service_worker/service_worker_context_request_handler_unittest.cc
@@ -217,6 +217,26 @@ TEST_P(ServiceWorkerContextRequestHandlerTestP,
EXPECT_TRUE(handler);
}
+// Tests that prefetches involving a ServiceWorker are dropped.
+TEST_P(ServiceWorkerContextRequestHandlerTestP, DropPrefetch) {
+ version_->SetStatus(ServiceWorkerVersion::NEW);
+ provider_host_->running_hosted_version_ = version_;
+
+ // Conduct a resource fetch for the main script.
+ const GURL kScriptUrl("http://host/script.js");
+ std::unique_ptr<net::URLRequest> request = url_request_context_.CreateRequest(
+ kScriptUrl, net::DEFAULT_PRIORITY, &url_request_delegate_);
+ request->SetLoadFlags(request->load_flags() | net::LOAD_PREFETCH);
+ std::unique_ptr<ServiceWorkerContextRequestHandler> handler(
+ new ServiceWorkerContextRequestHandler(
+ context()->AsWeakPtr(), provider_host_,
+ base::WeakPtr<storage::BlobStorageContext>(),
+ RESOURCE_TYPE_SERVICE_WORKER));
+ std::unique_ptr<net::URLRequestJob> job(
+ handler->MaybeCreateJob(request.get(), nullptr, nullptr));
+ EXPECT_FALSE(job.get());
+}
+
INSTANTIATE_TEST_CASE_P(ServiceWorkerContextRequestHandlerTest,
ServiceWorkerContextRequestHandlerTestP,
testing::Bool());

Powered by Google App Engine
This is Rietveld 408576698