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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/serviceworker/respond-with-response-body-with-invalid-chunk.html

Issue 2146773002: [Stream] Notify ReadObserver of Stream abortion (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: done Created 4 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: third_party/WebKit/LayoutTests/http/tests/serviceworker/respond-with-response-body-with-invalid-chunk.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/request-body-blob.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/respond-with-response-body-with-invalid-chunk.html
similarity index 50%
copy from third_party/WebKit/LayoutTests/http/tests/serviceworker/request-body-blob.html
copy to third_party/WebKit/LayoutTests/http/tests/serviceworker/respond-with-response-body-with-invalid-chunk.html
index 310281279368b7028978f4d1ef0d21d5f6b2ddf0..c06a0c1e05a36a16cf26c8f7c395718ba2faf4b3 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/request-body-blob.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/respond-with-response-body-with-invalid-chunk.html
@@ -1,10 +1,14 @@
<!DOCTYPE html>
<meta charset="utf-8">
-<title>request body</title>
-<script src="../resources/testharness.js"></script>
-<script src="../resources/testharnessreport.js"></script>
+<title>respondWith with response body having invalid chunks</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
<script src="resources/test-helpers.js"></script>
<script>
+const WORKER =
+ 'resources/respond-with-response-body-with-invalid-chunk-worker.js';
+const SCOPE =
+ 'resources/respond-with-response-body-with-invalid-chunk-iframe.html';
var resolve_done;
var done_was_called = new Promise(resolve => resolve_done = resolve);
// Called by the iframe when done.
@@ -12,17 +16,15 @@ function done(result) { resolve_done(result); }
// This test creates an controlled iframe that does a request with a request
// body blob. The service worker confirms it can read the blob and responds
falken 2016/07/14 13:00:43 This description doesn't seem to match this test.
yhirano 2016/07/14 13:30:54 Done.
-// with 'Pass'.
+// with 'PASS'.
promise_test(t => {
- var worker = 'resources/request-body-blob-worker.js';
- var scope = 'resources/request-body-blob-iframe.html';
- return service_worker_unregister_and_register(t, worker, scope)
+ return service_worker_unregister_and_register(t, WORKER, SCOPE)
.then(reg => {
add_completion_callback(() => reg.unregister());
return wait_for_state(t, reg.installing, 'activated');
})
- .then(() => with_iframe(scope))
+ .then(() => with_iframe(SCOPE))
.then(() => done_was_called)
- .then(result => assert_equals(result, 'Pass'));
- }, 'Service worker can read the request body blob');
+ .then(result => assert_equals(result, 'PASS'));
+ }, 'Response with a ReadableStream having non-Uint8Array chunks should be transferred as errored');
</script>
« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/respond-with-response-body-with-invalid-chunk-worker.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698