Allow redirects for requests that require preflight.

third_party/WebKit/LayoutTests/http/tests/serviceworker/resources/redirect.php

 <?php
-if (isset($_GET['Status'])) {
-  header ("HTTP/1.1 " . $_GET["Status"]);
-} else {
-  header ("HTTP/1.1 302");
-}
-$url = $_GET['Redirect'];
-if ($url != "noLocation") {
-  header("Location: $url");
+if ($_SERVER['REQUEST_METHOD'] !== 'OPTIONS') {

[yhirano, 2016/11/22 04:05:23]

Sorry, I don't understand the intention of this change. Can you explain?

[Jack Bates, 2016/11/22 18:24:17]

Without this, it sends a redirect response to the preflight request.
The tests that use REDIRECT_URL are for a redirect response to the actual
request (which is sometimes preceded by a preflight).

+  $url = $_GET['Redirect'];
+  if ($url != "noLocation") {
+    header("Location: $url");
+  }
+  if (isset($_GET['Status'])) {
+    header ("HTTP/1.1 " . $_GET["Status"]);
+  } else {
+    header ("HTTP/1.1 302");
+  }
 }
 if (isset($_GET['ACAOrigin'])) {
   $origins = explode(',', $_GET['ACAOrigin']);
   for ($i = 0; $i < sizeof($origins); ++$i)
     header("Access-Control-Allow-Origin: " . $origins[$i], false);
 }
 if (isset($_GET['ACAHeaders']))
     header("Access-Control-Allow-Headers: {$_GET['ACAHeaders']}");
 if (isset($_GET['ACAMethods']))
     header("Access-Control-Allow-Methods: {$_GET['ACAMethods']}");
 if (isset($_GET['ACACredentials']))
     header("Access-Control-Allow-Credentials: {$_GET['ACACredentials']}");
 if (isset($_GET['ACEHeaders']))
     header("Access-Control-Expose-Headers: {$_GET['ACEHeaders']}");
 if (isset($_GET['NoRedirectTest'])) {
     echo "report({jsonpResult:'noredirect'});";
 }
 ?>