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

Unified Diff: chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc

Issue 2575523002: Prerender: Confirm ServiceWorkers are invoked for NoState Prefetch (Closed)
Patch Set: check sw exist 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
« no previous file with comments | « chrome/browser/prerender/prerender_final_status.cc ('k') | chrome/test/data/prerender/service_worker.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc
diff --git a/chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc b/chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc
index 6c1d72ff9bc201bdcb292665c6110df47712c3d7..906aba7456f26f76e14ddd5c1808f95ffc40c57e 100644
--- a/chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc
+++ b/chrome/browser/prerender/prerender_nostate_prefetch_browsertest.cc
@@ -49,6 +49,7 @@ const char kPrefetchResponseHeaderCSP[] =
"prerender/prefetch_response_csp.html";
const char kPrefetchScript[] = "prerender/prefetch.js";
const char kPrefetchScript2[] = "prerender/prefetch2.js";
+const char kServiceWorkerLoader[] = "prerender/service_worker.html";
const char kPrefetchSubresourceRedirectPage[] =
"prerender/prefetch_subresource_redirect.html";
@@ -514,4 +515,24 @@ IN_PROC_BROWSER_TEST_F(NoStatePrefetchBrowserTest, IssuesIdlePriorityRequests) {
script_counter.WaitForCount(1);
}
+// Checks that prefetch is canceled when there is a ServiceWorker.
+IN_PROC_BROWSER_TEST_F(NoStatePrefetchBrowserTest, CancelOnServiceWorker) {
+ ui_test_utils::NavigateToURL(
+ current_browser(),
+ src_server()->GetURL(MakeAbsolute(kServiceWorkerLoader)));
+
+ PrefetchFromFile(kPrefetchPage, FINAL_STATUS_SERVICE_WORKER);
+}
+
+IN_PROC_BROWSER_TEST_F(NoStatePrefetchBrowserTest, CheckNoSW) {
mattcary 2016/12/13 16:08:47 This test here shows that a SW actually exists: I
+ ui_test_utils::NavigateToURL(
+ current_browser(),
+ src_server()->GetURL(MakeAbsolute(kServiceWorkerLoader)));
+
+ RequestCounter png;
+ CountRequestFor(kPrefetchPng, &png);
+ PrefetchFromFile(kPrefetchPage, FINAL_STATUS_SERVICE_WORKER);
+ png.WaitForCount(1);
+}
+
} // namespace prerender
« no previous file with comments | « chrome/browser/prerender/prerender_final_status.cc ('k') | chrome/test/data/prerender/service_worker.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698