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

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: fix 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 56%
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..e223c0b4cb94027642841e5288316b21694df56e 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.
@@ -14,15 +18,13 @@ function done(result) { resolve_done(result); }
// body blob. The service worker confirms it can read the blob and responds
// 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'));
tyoshino (SeeGerritForStatus) 2016/07/13 06:09:35 PASS?
yhirano 2016/07/13 10:35:56 Done.
- }, 'Service worker can read the request body blob');
+ }, 'Response with a ReadableStream having non-Uint8Array chunks should be transferred as errored');
</script>

Powered by Google App Engine
This is Rietveld 408576698