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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php

Issue 2438723002: Add layout tests about sendBeacon's handling of credentials for cross origin redirects
Patch Set: Created 4 years, 2 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 | « third_party/WebKit/LayoutTests/http/tests/navigation/beacon-cross-origin-redirect-credentials.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php
diff --git a/third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php b/third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php
index a43ff62c29929a8604040105ebf7b80ebe87d535..1888c9329cc5bdbd380cf21a39a513310e89488e 100644
--- a/third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php
+++ b/third_party/WebKit/LayoutTests/http/tests/navigation/resources/save-beacon.php
@@ -32,6 +32,9 @@ if (strpos($contentType, "application/") !== false) {
fwrite($beaconFile, "Body: $postdata\n");
fclose($beaconFile);
rename($beaconFilename . ".tmp", $beaconFilename);
-foreach ($_COOKIE as $name => $value)
- setcookie($name, "deleted", time() - 60, "/");
+
+if (!array_key_exists('dontclearcookies', $_GET)) {
+ foreach ($_COOKIE as $name => $value)
+ setcookie($name, "deleted", time() - 60, "/");
+}
?>
« no previous file with comments | « third_party/WebKit/LayoutTests/http/tests/navigation/beacon-cross-origin-redirect-credentials.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698