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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html

Issue 2702113003: Move tests for sendBeacon to a separate dedicated directory for clearer ownership (Closed)
Patch Set: a Created 3 years, 10 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/sendbeacon/beacon-cross-origin-redirect-blob.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/navigation/beacon-cross-origin-redirect-blob.html b/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html
similarity index 71%
rename from third_party/WebKit/LayoutTests/http/tests/navigation/beacon-cross-origin-redirect-blob.html
rename to third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html
index 1d8ce2bc03f9a787d4a7415d8c7ebe9b8fef4f4f..6fa52344eeb0c5098c708edc82737f13891a30bc 100644
--- a/third_party/WebKit/LayoutTests/http/tests/navigation/beacon-cross-origin-redirect-blob.html
+++ b/third_party/WebKit/LayoutTests/http/tests/sendbeacon/beacon-cross-origin-redirect-blob.html
@@ -15,11 +15,12 @@ function test() {
}
blob = new Blob(["Cross", "Origin"], {type: "text/plain;from-beacon=true"});
- shouldBeTrue('navigator.sendBeacon("http://127.0.0.1:8080/navigation/resources/redirection-response.php?status=302&simple=true&target=non-existent.php", blob);');
+ // The "simple" parameter is just for differentiating the URLs.
+ shouldBeTrue('navigator.sendBeacon("http://127.0.0.1:8080/resources/redirection-response.php?status=302&simple=true&target=/non-existent.php", blob);');
// Wait a while for the redirect response handling to happen before finishing up.
setTimeout(function () {
blob = new Blob([new Uint8Array(20)], {type: "application/octet-stream"});
- shouldBeTrue('navigator.sendBeacon("http://127.0.0.1:8080/navigation/resources/redirection-response.php?status=302&simple=false&target=non-existent.php", blob);');
+ shouldBeTrue('navigator.sendBeacon("http://127.0.0.1:8080/resources/redirection-response.php?status=302&simple=false&target=/non-existent.php", blob);');
setTimeout(finishJSTest, 200);
}, 200);
}

Powered by Google App Engine
This is Rietveld 408576698