Correct SOUP and cURL builds after r221017.
authorbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 21:54:40 +0000 (21:54 +0000)
committerbfulgham@apple.com <bfulgham@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Aug 2017 21:54:40 +0000 (21:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=175846

Reviewed by Michael Catanzaro.

Correct the method signatures for 'cookiesForDOM' on the SOUP and cURL backends.
Note that these ports will need to add specific logic to find/filter secure cookies
if requested by the caller.

* platform/network/curl/CookieJarCurl.cpp:
(WebCore::CookieJarCurlFileSystem::cookiesForDOM):
(WebCore::cookiesForDOM):
* platform/network/curl/CookieJarCurl.h:
* platform/network/soup/CookieJarSoup.cpp:
(WebCore::cookiesForDOM):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@221050 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/network/curl/CookieJarCurl.cpp
Source/WebCore/platform/network/curl/CookieJarCurl.h
Source/WebCore/platform/network/soup/CookieJarSoup.cpp

index 7290414..b65ee9c 100644 (file)
@@ -1,3 +1,21 @@
+2017-08-22  Brent Fulgham  <bfulgham@apple.com>
+
+        Correct SOUP and cURL builds after r221017.
+        https://bugs.webkit.org/show_bug.cgi?id=175846
+
+        Reviewed by Michael Catanzaro.
+
+        Correct the method signatures for 'cookiesForDOM' on the SOUP and cURL backends.
+        Note that these ports will need to add specific logic to find/filter secure cookies
+        if requested by the caller.
+
+        * platform/network/curl/CookieJarCurl.cpp:
+        (WebCore::CookieJarCurlFileSystem::cookiesForDOM):
+        (WebCore::cookiesForDOM):
+        * platform/network/curl/CookieJarCurl.h:
+        * platform/network/soup/CookieJarSoup.cpp:
+        (WebCore::cookiesForDOM):
+
 2017-08-22  Chris Dumez  <cdumez@apple.com>
 
         [EnabledBySetting] in WebIDL uses Document but does not include Document.h
index 2b7de47..791fdcd 100644 (file)
@@ -287,9 +287,10 @@ static String cookiesForSession(const NetworkStorageSession&, const URL&, const
     return cookies;
 }
 
-String CookieJarCurlFileSystem::cookiesForDOM(const NetworkStorageSession& session, const URL& firstParty, const URL& url)
+std::pair<String, bool> CookieJarCurlFileSystem::cookiesForDOM(const NetworkStorageSession& session, const URL& firstParty, const URL& url, IncludeSecureCookies)
 {
-    return cookiesForSession(session, firstParty, url, false);
+    // FIXME: This should filter secure cookies out if the caller requests it.
+    return { cookiesForSession(session, firstParty, url, false), false };
 }
 
 String CookieJarCurlFileSystem::cookieRequestHeaderFieldValue(const NetworkStorageSession& session, const URL& firstParty, const URL& url)
@@ -336,9 +337,9 @@ void CookieJarCurlFileSystem::deleteAllCookiesModifiedSince(const NetworkStorage
 
 // dispatcher functions
 
-String cookiesForDOM(const NetworkStorageSession& session, const URL& firstParty, const URL& url)
+std::pair<String, bool> cookiesForDOM(const NetworkStorageSession& session, const URL& firstParty, const URL& url, IncludeSecureCookies includeSecureCookies)
 {
-    return CurlContext::singleton().cookieJar().cookiesForDOM(session, firstParty, url);
+    return CurlContext::singleton().cookieJar().cookiesForDOM(session, firstParty, url, includeSecureCookies);
 }
 
 void setCookiesFromDOM(const NetworkStorageSession& session, const URL& firstParty, const URL& url, const String& value)
index 6684572..7156831 100644 (file)
@@ -28,7 +28,7 @@ class URL;
 
 class CookieJarCurl {
 public:
-    virtual String cookiesForDOM(const NetworkStorageSession&, const URL& firstParty, const URL&) = 0;
+    virtual std::pair<String, bool> cookiesForDOM(const NetworkStorageSession&, const URL& firstParty, const URL&, IncludeSecureCookies) = 0;
     virtual void setCookiesFromDOM(const NetworkStorageSession&, const URL& firstParty, const URL&, const String&) = 0;
     virtual bool cookiesEnabled(const NetworkStorageSession&, const URL& firstParty, const URL&) = 0;
     virtual String cookieRequestHeaderFieldValue(const NetworkStorageSession&, const URL& firstParty, const URL&) = 0;
@@ -41,7 +41,7 @@ public:
 };
 
 class CookieJarCurlFileSystem : public CookieJarCurl {
-    String cookiesForDOM(const NetworkStorageSession&, const URL& firstParty, const URL&) override;
+    std::pair<String, bool> cookiesForDOM(const NetworkStorageSession&, const URL& firstParty, const URL&, IncludeSecureCookies) override;
     void setCookiesFromDOM(const NetworkStorageSession&, const URL& firstParty, const URL&, const String&) override;
     bool cookiesEnabled(const NetworkStorageSession&, const URL& firstParty, const URL&) override;
     String cookieRequestHeaderFieldValue(const NetworkStorageSession&, const URL& firstParty, const URL&) override;
index 2425498..f8b215b 100644 (file)
@@ -88,8 +88,9 @@ static String cookiesForSession(const NetworkStorageSession& session, const URL&
     return String::fromUTF8(cookies.get());
 }
 
-String cookiesForDOM(const NetworkStorageSession& session, const URL&, const URL& url)
+std::pair<String, bool> cookiesForDOM(const NetworkStorageSession& session, const URL&, const URL& url, IncludeSecureCookies)
 {
+    // FIXME: SOUP concept of secure cookies should be filtered here.
     return cookiesForSession(session, url, false);
 }