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

Unified Diff: LayoutTests/http/tests/serviceworker/chromium/force-refresh-ready.html

Issue 1184923005: ServiceWorker: Add layout test for ready promise resolve after shift-reload. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 years, 6 months 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/http/tests/serviceworker/chromium/force-refresh-ready.html
diff --git a/LayoutTests/http/tests/serviceworker/chromium/force-refresh-ready.html b/LayoutTests/http/tests/serviceworker/chromium/force-refresh-ready.html
new file mode 100644
index 0000000000000000000000000000000000000000..f825b9b590db35872f6aea8e3bcd8cf7376a25d9
--- /dev/null
+++ b/LayoutTests/http/tests/serviceworker/chromium/force-refresh-ready.html
@@ -0,0 +1,42 @@
+<!DOCTYPE html>
+<!-- This test should not be upstreamed due to shift-reload behavior
+is not defined by spec. -->
+<title>Service Worker: force-refresh ready test</title>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharness-helpers.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script src="../resources/test-helpers.js"></script>
+<body>
+<script>
+
+promise_test(function(t) {
+ var scope = '../resources/blank.html';
+ var script_url = '../resources/empty-worker.js';
+ var frame;
+ return service_worker_unregister_and_register(t, script_url, scope)
+ .then(function(registration) {
+ return wait_for_state(t, registration.installing, 'activated');
+ })
+ .then(function() {
+ return with_iframe(scope);
+ })
+ .then(function(f) {
+ frame = f;
+ return new Promise(function(resolve) {
+ frame.onload = t.step_func(function() {
+ frame.contentWindow.navigator.serviceWorker.ready
+ .then(function() {
+ resolve();
+ });
+ });
+ frame.contentWindow.internals.forceReload(true);
+ });
+ })
+ .then(function() {
+ frame.remove();
+ return service_worker_unregister_and_done(t, scope);
+ });
+ }, 'Test ready promise for force-refreshed frame');
+
+</script>
+</body>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698