Title: [93188] trunk/Source/WebCore
Revision
93188
Author
le...@chromium.org
Date
2011-08-16 18:32:20 -0700 (Tue, 16 Aug 2011)

Log Message

REGRESSION (r89086): All worker xhr requests trigger preflight requests.
https://bugs.webkit.org/show_bug.cgi?id=66340

Reviewed by Adam Barth.

Test coming in a more comprehensive patch shortly. This is just a very targetted
patch to allow for easier merging.

* loader/CrossOriginAccessControl.cpp:
(WebCore::isOnAccessControlSimpleRequestHeaderWhitelist): Add referer to the list
of whitelisted headers. It can't be set by xhr so it is ok to whitelist it, but it is
set before calling this function. This is similar to what is done for "origin" in here.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (93187 => 93188)


--- trunk/Source/WebCore/ChangeLog	2011-08-17 01:29:17 UTC (rev 93187)
+++ trunk/Source/WebCore/ChangeLog	2011-08-17 01:32:20 UTC (rev 93188)
@@ -1,3 +1,18 @@
+2011-08-16  David Levin  <le...@chromium.org>
+
+        REGRESSION (r89086): All worker xhr requests trigger preflight requests.
+        https://bugs.webkit.org/show_bug.cgi?id=66340
+
+        Reviewed by Adam Barth.
+
+        Test coming in a more comprehensive patch shortly. This is just a very targetted
+        patch to allow for easier merging.
+
+        * loader/CrossOriginAccessControl.cpp:
+        (WebCore::isOnAccessControlSimpleRequestHeaderWhitelist): Add referer to the list
+        of whitelisted headers. It can't be set by xhr so it is ok to whitelist it, but it is
+        set before calling this function. This is similar to what is done for "origin" in here.
+
 2011-08-16  Shawn Singh  <shawnsi...@chromium.org>
 
         Computing screen-space transform for LayerChromium and CCLayerImpl

Modified: trunk/Source/WebCore/loader/CrossOriginAccessControl.cpp (93187 => 93188)


--- trunk/Source/WebCore/loader/CrossOriginAccessControl.cpp	2011-08-17 01:29:17 UTC (rev 93187)
+++ trunk/Source/WebCore/loader/CrossOriginAccessControl.cpp	2011-08-17 01:32:20 UTC (rev 93188)
@@ -46,7 +46,8 @@
     if (equalIgnoringCase(name, "accept")
         || equalIgnoringCase(name, "accept-language")
         || equalIgnoringCase(name, "content-language")
-        || equalIgnoringCase(name, "origin"))
+        || equalIgnoringCase(name, "origin")
+        || equalIgnoringCase(name, "referer"))
         return true;
 
     // Preflight is required for MIME types that can not be sent via form submission.
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to