Title: [151447] trunk/Source/WebKit2
Revision
151447
Author
jer.no...@apple.com
Date
2013-06-11 11:48:08 -0700 (Tue, 11 Jun 2013)

Log Message

Do not filter cookies requested through the CookieStorageShim.
https://bugs.webkit.org/show_bug.cgi?id=117511
<rdar://problem/13603433>

Reviewed by Beth Dakin.

Since the shim function is called when the underlying media engine is
creating an HTTPRequest, use CookieRequestHeaderFieldValue (which does
not filter out HTTP-only cookies) rather than CookiesForDOM (which does).

* Shared/mac/CookieStorageShim.cpp:
(WebKit::webKitCookieStorageCopyRequestHeaderFieldsForURL):

Modified Paths

Diff

Modified: trunk/Source/WebKit2/ChangeLog (151446 => 151447)


--- trunk/Source/WebKit2/ChangeLog	2013-06-11 18:33:36 UTC (rev 151446)
+++ trunk/Source/WebKit2/ChangeLog	2013-06-11 18:48:08 UTC (rev 151447)
@@ -1,3 +1,18 @@
+2013-06-11  Jer Noble  <jer.no...@apple.com>
+
+        Do not filter cookies requested through the CookieStorageShim.
+        https://bugs.webkit.org/show_bug.cgi?id=117511
+        <rdar://problem/13603433>
+
+        Reviewed by Beth Dakin.
+
+        Since the shim function is called when the underlying media engine is
+        creating an HTTPRequest, use CookieRequestHeaderFieldValue (which does
+        not filter out HTTP-only cookies) rather than CookiesForDOM (which does).
+
+        * Shared/mac/CookieStorageShim.cpp:
+        (WebKit::webKitCookieStorageCopyRequestHeaderFieldsForURL):
+
 2013-06-11  Seokju Kwon  <seokju.k...@gmail.com>
 
         Remove leftover wxWebkit code

Modified: trunk/Source/WebKit2/Shared/mac/CookieStorageShim.cpp (151446 => 151447)


--- trunk/Source/WebKit2/Shared/mac/CookieStorageShim.cpp	2013-06-11 18:33:36 UTC (rev 151446)
+++ trunk/Source/WebKit2/Shared/mac/CookieStorageShim.cpp	2013-06-11 18:48:08 UTC (rev 151447)
@@ -47,7 +47,7 @@
 {
     String cookies;
     KURL firstPartyForCookiesURL;
-    if (!WebProcess::shared().networkConnection()->connection()->sendSync(Messages::NetworkConnectionToWebProcess::CookiesForDOM(false, firstPartyForCookiesURL, inRequestURL), Messages::NetworkConnectionToWebProcess::CookiesForDOM::Reply(cookies), 0))
+    if (!WebProcess::shared().networkConnection()->connection()->sendSync(Messages::NetworkConnectionToWebProcess::CookieRequestHeaderFieldValue(false, firstPartyForCookiesURL, inRequestURL), Messages::NetworkConnectionToWebProcess::CookiesForDOM::Reply(cookies), 0))
         return 0;
 
     RetainPtr<CFStringRef> cfCookies = cookies.createCFString();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to