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

Unified Diff: LayoutTests/http/tests/serviceworker/fetch-event-respond-with-argument.html

Issue 1255933004: Service Worker: Correct Web IDL of FetchEvent.respondWith method. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rename IDL argument and method params. Created 5 years, 5 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
Index: LayoutTests/http/tests/serviceworker/fetch-event-respond-with-argument.html
diff --git a/LayoutTests/http/tests/serviceworker/fetch-event-network-error.html b/LayoutTests/http/tests/serviceworker/fetch-event-respond-with-argument.html
similarity index 73%
copy from LayoutTests/http/tests/serviceworker/fetch-event-network-error.html
copy to LayoutTests/http/tests/serviceworker/fetch-event-respond-with-argument.html
index 567ffc14dfa218dc9807f8a283f42e326594e4b8..ba69cc76b5e5282efff70e71971e458f75b7b4a4 100644
--- a/LayoutTests/http/tests/serviceworker/fetch-event-network-error.html
+++ b/LayoutTests/http/tests/serviceworker/fetch-event-respond-with-argument.html
@@ -1,5 +1,5 @@
<!DOCTYPE html>
-<title>Service Worker: Fetch event network error</title>
+<title>Service Worker: FetchEvent.respondWith() argument type test.</title>
<script src="../resources/testharness.js"></script>
<script src="../resources/testharness-helpers.js"></script>
<script src="../resources/testharnessreport.js"></script>
@@ -17,8 +17,8 @@ function notify_test_done(result) {
}
promise_test(function(t) {
- var scope = 'resources/fetch-event-network-error-controllee-iframe.html';
- var script = 'resources/fetch-event-network-error-worker.js';
+ var scope = 'resources/fetch-event-respond-with-argument-iframe.html';
+ var script = 'resources/fetch-event-respond-with-argument-worker.js';
var frame;
return service_worker_unregister_and_register(t, script, scope)
@@ -37,6 +37,6 @@ promise_test(function(t) {
assert_equals(result, 'PASS');
return service_worker_unregister_and_done(t, scope);
});
- }, 'Rejecting the fetch event or using preventDefault() causes a network ' +
- 'error');
+ }, 'respondWith() takes either a Response or a promise that resolves ' +
+ 'with a Response. Other values should raise a network error.');
</script>

Powered by Google App Engine
This is Rietveld 408576698