Attachment 171282 Details for Bug 99340 – WIP Patch

[patch] WIP Patch

bug-99340-20121029141921.patch (text/plain), 47.06 KB, created by Mark Pilgrim (Google) on 2012-10-29 11:21:00 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Pilgrim (Google)

Created: 2012-10-29 11:21:00 PDT

Size: 47.06 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 132828)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,109 @@
>+2012-10-29  Mark Pilgrim  <pilgrim@chromium.org>
>+
>+        [Chromium] Remove cookie-related functions from PlatformSupport
>+        https://bugs.webkit.org/show_bug.cgi?id=99340
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Additional information of the change such as approach, rationale. Please add per-function descriptions below (OOPS!).
>+
>+        No new tests (OOPS!).
>+
>+        * Modules/websockets/WebSocketChannel.cpp:
>+        (WebCore::WebSocketChannel::processBuffer):
>+        * Modules/websockets/WebSocketHandshake.cpp:
>+        (WebCore::WebSocketHandshake::clientHandshakeMessage):
>+        (WebCore::WebSocketHandshake::clientHandshakeRequest):
>+        * dom/Document.cpp:
>+        (WebCore::Document::cookie):
>+        (WebCore::Document::setCookie):
>+        (WebCore):
>+        (WebCore::Document::cookiesEnabled):
>+        * dom/Document.h:
>+        (Document):
>+        * inspector/InspectorPageAgent.cpp:
>+        (WebCore::InspectorPageAgent::getCookies):
>+        (WebCore::InspectorPageAgent::deleteCookie):
>+        * loader/FrameLoaderClient.h:
>+        (WebCore):
>+        (FrameLoaderClient):
>+        (WebCore::FrameLoaderClient::cookieJar):
>+        * page/Navigator.cpp:
>+        (WebCore::Navigator::cookieEnabled):
>+        * platform/CookieJar.h:
>+        (WebCore):
>+        (CookieJar):
>+        (WebCore::CookieJar::CookieJar):
>+        * platform/chromium/PlatformSupport.h:
>+        (PlatformSupport):
>+        * platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
>+        (WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime):
>+        * platform/mac/CookieJar.mm:
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * platform/network/cf/CookieJarCFNet.cpp:
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * platform/network/chromium/CookieJarChromium.cpp:
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * platform/network/curl/CookieJarCurl.cpp:
>+        (WebCore):
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * platform/network/win/CookieJarWin.cpp:
>+        (WebCore):
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * platform/qt/CookieJarQt.cpp:
>+        (WebCore::CookieJar::setCookies):
>+        (WebCore::CookieJar::cookies):
>+        (WebCore::CookieJar::cookieRequestHeaderFieldValue):
>+        (WebCore::CookieJar::cookiesEnabled):
>+        (WebCore::CookieJar::getRawCookies):
>+        (WebCore::CookieJar::deleteCookie):
>+        (WebCore::CookieJar::getHostnamesWithCookies):
>+        (WebCore::CookieJar::deleteCookiesForHostname):
>+        (WebCore::CookieJar::deleteAllCookies):
>+        * plugins/PluginView.cpp:
>+        (WebCore::PluginView::getValueForURL):
>+        (WebCore::PluginView::setValueForURL):
>+
> 2012-10-29  Arnaud Renevier  <a.renevier@sisa.samsung.com>
> 
>         webview not redrawn as needed when accelerated compositing is enabled.
>Index: Source/WebCore/Modules/websockets/WebSocketChannel.cpp
>===================================================================
>--- Source/WebCore/Modules/websockets/WebSocketChannel.cpp	(revision 132793)
>+++ Source/WebCore/Modules/websockets/WebSocketChannel.cpp	(working copy)
>@@ -430,7 +430,7 @@ bool WebSocketChannel::processBuffer()
>             if (m_identifier)
>                 InspectorInstrumentation::didReceiveWebSocketHandshakeResponse(m_document, m_identifier, m_handshake->serverHandshakeResponse());
>             if (!m_handshake->serverSetCookie().isEmpty()) {
>-                if (cookiesEnabled(m_document)) {
>+                if (m_document->cookiesEnabled()) {
>                     ExceptionCode ec; // Exception (for sandboxed documents) ignored.
>                     m_document->setCookie(m_handshake->serverSetCookie(), ec);
>                 }
>Index: Source/WebCore/Modules/websockets/WebSocketHandshake.cpp
>===================================================================
>--- Source/WebCore/Modules/websockets/WebSocketHandshake.cpp	(revision 132793)
>+++ Source/WebCore/Modules/websockets/WebSocketHandshake.cpp	(working copy)
>@@ -39,6 +39,9 @@
> #include "Cookie.h"
> #include "CookieJar.h"
> #include "Document.h"
>+#include "Frame.h"
>+#include "FrameLoader.h"
>+#include "FrameLoaderClient.h"
> #include "HTTPHeaderMap.h"
> #include "HTTPParsers.h"
> #include "KURL.h"
>@@ -200,7 +203,10 @@ CString WebSocketHandshake::clientHandsh
>     KURL url = httpURLForAuthenticationAndCookies();
>     if (m_context->isDocument()) {
>         Document* document = static_cast<Document*>(m_context);
>-        String cookie = cookieRequestHeaderFieldValue(document, url);
>+        String cookie;
>+        Frame* frame = document->frame();
>+        if (frame)
>+            cookie = frame->loader()->client()->cookieJar().cookieRequestHeaderFieldValue(url, document->firstPartyForCookies());
>         if (!cookie.isEmpty())
>             fields.append("Cookie: " + cookie);
>         // Set "Cookie2: <cookie>" if cookies 2 exists for url?
>@@ -249,7 +255,12 @@ PassRefPtr<WebSocketHandshakeRequest> We
>     KURL url = httpURLForAuthenticationAndCookies();
>     if (m_context->isDocument()) {
>         Document* document = static_cast<Document*>(m_context);
>-        String cookie = cookieRequestHeaderFieldValue(document, url);
>+        String cookie;
>+        Frame* frame = document->frame();
>+        if (frame) {
>+            CookieJar cookieJar = frame->loader()->client()->cookieJar();
>+            cookie = cookieJar.cookieRequestHeaderFieldValue(url, document->firstPartyForCookies());
>+        }
>         if (!cookie.isEmpty())
>             request->addHeaderField("Cookie", cookie);
>         // Set "Cookie2: <cookie>" if cookies 2 exists for url?
>Index: Source/WebCore/dom/Document.cpp
>===================================================================
>--- Source/WebCore/dom/Document.cpp	(revision 132793)
>+++ Source/WebCore/dom/Document.cpp	(working copy)
>@@ -3816,7 +3816,9 @@ String Document::cookie(ExceptionCode& e
>     if (cookieURL.isEmpty())
>         return String();
> 
>-    return cookies(this, cookieURL);
>+    if (!frame())
>+        return String();
>+    return frame()->loader()->client()->cookieJar().cookies(cookieURL, this->firstPartyForCookies());
> }
> 
> void Document::setCookie(const String& value, ExceptionCode& ec)
>@@ -3837,7 +3839,23 @@ void Document::setCookie(const String& v
>     if (cookieURL.isEmpty())
>         return;
> 
>-    setCookies(this, cookieURL, value);
>+    if (!frame())
>+        return;
>+    frame()->loader()->client()->cookieJar().setCookies(cookieURL, this->firstPartyForCookies(), value);
>+}
>+
>+bool Document::cookiesEnabled()
>+{
>+    if (!page())
>+        return false;
>+
>+    if (!page()->settings()->cookieEnabled())
>+        return false;
>+
>+    if (!frame())
>+        return false;
>+
>+    return frame()->loader()->client()->cookieJar().cookiesEnabled(this->cookieURL(), this->firstPartyForCookies());
> }
> 
> String Document::referrer() const
>Index: Source/WebCore/dom/Document.h
>===================================================================
>--- Source/WebCore/dom/Document.h	(revision 132793)
>+++ Source/WebCore/dom/Document.h	(working copy)
>@@ -817,6 +817,7 @@ public:
> 
>     String cookie(ExceptionCode&) const;
>     void setCookie(const String&, ExceptionCode&);
>+    bool cookiesEnabled();
> 
>     String referrer() const;
> 
>Index: Source/WebCore/inspector/InspectorPageAgent.cpp
>===================================================================
>--- Source/WebCore/inspector/InspectorPageAgent.cpp	(revision 132793)
>+++ Source/WebCore/inspector/InspectorPageAgent.cpp	(working copy)
>@@ -49,6 +49,8 @@
> #include "Document.h"
> #include "DocumentLoader.h"
> #include "Frame.h"
>+#include "FrameLoader.h"
>+#include "FrameLoaderClient.h"
> #include "FrameView.h"
> #include "GeolocationController.h"
> #include "GeolocationError.h"
>@@ -512,9 +514,10 @@ void InspectorPageAgent::getCookies(Erro
>     for (Frame* frame = mainFrame(); frame; frame = frame->tree()->traverseNext(mainFrame())) {
>         Document* document = frame->document();
>         Vector<KURL> allURLs = allResourcesURLsForFrame(frame);
>+        CookieJar cookieJar = frame->loader()->client()->cookieJar();
>         for (Vector<KURL>::const_iterator it = allURLs.begin(); it != allURLs.end(); ++it) {
>             Vector<Cookie> docCookiesList;
>-            rawCookiesImplemented = getRawCookies(document, KURL(ParsedURLString, *it), docCookiesList);
>+            rawCookiesImplemented = cookieJar.getRawCookies(KURL(ParsedURLString, *it), document->firstPartyForCookies(), docCookiesList);
>             if (!rawCookiesImplemented) {
>                 // FIXME: We need duplication checking for the String representation of cookies.
>                 ExceptionCode ec = 0;
>@@ -550,8 +553,9 @@ void InspectorPageAgent::deleteCookie(Er
>             continue;
> 
>         Vector<KURL> allURLs = allResourcesURLsForFrame(frame);
>+        CookieJar cookieJar = frame->loader()->client()->cookieJar();
>         for (Vector<KURL>::const_iterator it = allURLs.begin(); it != allURLs.end(); ++it)
>-            WebCore::deleteCookie(document, KURL(ParsedURLString, *it), cookieName);
>+            cookieJar.deleteCookie(KURL(ParsedURLString, *it), cookieName);
>     }
> }
> 
>Index: Source/WebCore/loader/FrameLoaderClient.h
>===================================================================
>--- Source/WebCore/loader/FrameLoaderClient.h	(revision 132793)
>+++ Source/WebCore/loader/FrameLoaderClient.h	(working copy)
>@@ -30,6 +30,7 @@
> #ifndef FrameLoaderClient_h
> #define FrameLoaderClient_h
> 
>+#include "CookieJar.h"
> #include "FrameLoaderTypes.h"
> #include "IconURL.h"
> #include "LayoutMilestones.h"
>@@ -64,6 +65,7 @@ namespace WebCore {
>     class AuthenticationChallenge;
>     class CachedFrame;
>     class Color;
>+    class CookieJar;
>     class DOMWindowExtension;
>     class DOMWrapperWorld;
>     class DocumentLoader;
>@@ -349,6 +351,8 @@ namespace WebCore {
> #if ENABLE(MEDIA_STREAM)
>         virtual void dispatchWillStartUsingPeerConnectionHandler(RTCPeerConnectionHandler*) { }
> #endif
>+
>+        virtual CookieJar cookieJar() { return CookieJar(); }
>     };
> 
> } // namespace WebCore
>Index: Source/WebCore/page/Navigator.cpp
>===================================================================
>--- Source/WebCore/page/Navigator.cpp	(revision 132793)
>+++ Source/WebCore/page/Navigator.cpp	(working copy)
>@@ -23,7 +23,6 @@
> #include "config.h"
> #include "Navigator.h"
> 
>-#include "CookieJar.h"
> #include "DOMMimeTypeArray.h"
> #include "DOMPluginArray.h"
> #include "Document.h"
>@@ -118,7 +117,7 @@ bool Navigator::cookieEnabled() const
>     if (m_frame->page() && !m_frame->page()->settings()->cookieEnabled())
>         return false;
> 
>-    return cookiesEnabled(m_frame->document());
>+    return m_frame->document()->cookiesEnabled();
> }
> 
> bool Navigator::javaEnabled() const
>Index: Source/WebCore/platform/CookieJar.h
>===================================================================
>--- Source/WebCore/platform/CookieJar.h	(revision 132793)
>+++ Source/WebCore/platform/CookieJar.h	(working copy)
>@@ -26,6 +26,13 @@
> #ifndef CookieJar_h
> #define CookieJar_h
> 
>+#if PLATFORM(CHROMIUM)
>+#include <public/WebCookieJar.h>
>+#endif
>+#if PLATFORM(QT)
>+#include "NetworkingContext.h"
>+#endif
>+
> #include <wtf/Forward.h>
> #include <wtf/HashSet.h>
> #include <wtf/text/WTFString.h>
>@@ -33,23 +40,54 @@
> 
> namespace WebCore {
> 
>-    class Document;
>-    class KURL;
>+class KURL;
>+struct Cookie;
> 
>-    struct Cookie;
>+class CookieJar {
>+public:
>+    CookieJar()
>+    {
>+#if PLATFORM(CHROMIUM)
>+        m_webCookieJar = 0;
>+#endif
>+#if PLATFORM(QT)
>+        m_networkingContext = 0;
>+#endif
>+    };
>+#if PLATFORM(CHROMIUM)
>+    explicit CookieJar(WebKit::WebCookieJar* webCookieJar)
>+    {
>+        m_webCookieJar = webCookieJar;
>+    };
>+#endif
>+#if PLATFORM(QT)
>+    explicit CookieJar(NetworkingContext* context)
>+    {
>+        m_networkingContext = context;
>+    };
>+#endif
> 
>     // cookies omits HttpOnly cookies.
>-    String cookies(const Document*, const KURL&);
>-    String cookieRequestHeaderFieldValue(const Document*, const KURL&);
>-    void setCookies(Document*, const KURL&, const String&);
>-    bool cookiesEnabled(const Document*);
>-    bool getRawCookies(const Document*, const KURL&, Vector<Cookie>&);
>-    void deleteCookie(const Document*, const KURL&, const String&);
>+    String cookies(const KURL&, const KURL&);
>+    String cookieRequestHeaderFieldValue(const KURL&, const KURL&);
>+    void setCookies(const KURL&, const KURL&, const String&);
>+    bool cookiesEnabled(const KURL&, const KURL&);
>+    bool getRawCookies(const KURL&, const KURL&, Vector<Cookie>&);
>+    void deleteCookie(const KURL&, const String&);
> 
>     void getHostnamesWithCookies(HashSet<String>& hostnames);
>     void deleteCookiesForHostname(const String& hostname);
>     void deleteAllCookies();
> 
>-}
>+private:
>+#if PLATFORM(CHROMIUM)
>+    WebKit::WebCookieJar* m_webCookieJar;
>+#endif
>+#if PLATFORM(QT)
>+    NetworkingContext* m_networkingContext;
>+#endif
>+};
>+
>+} // namespace WebCore
> 
> #endif
>Index: Source/WebCore/platform/chromium/PlatformSupport.h
>===================================================================
>--- Source/WebCore/platform/chromium/PlatformSupport.h	(revision 132793)
>+++ Source/WebCore/platform/chromium/PlatformSupport.h	(working copy)
>@@ -78,14 +78,6 @@ struct FontRenderStyle;
> 
> class PlatformSupport {
> public:
>-    // Cookies ------------------------------------------------------------
>-    static void setCookies(const Document*, const KURL&, const String& value);
>-    static String cookies(const Document*, const KURL&);
>-    static String cookieRequestHeaderFieldValue(const Document*, const KURL&);
>-    static bool rawCookies(const Document*, const KURL&, Vector<Cookie>&);
>-    static void deleteCookie(const Document*, const KURL&, const String& cookieName);
>-    static bool cookiesEnabled(const Document*);
>-
>     // Font ---------------------------------------------------------------
> #if OS(WINDOWS)
>     static bool ensureFontLoaded(HFONT);
>Index: Source/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp
>===================================================================
>--- Source/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp	(revision 132793)
>+++ Source/WebCore/platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp	(working copy)
>@@ -32,6 +32,8 @@
> #include "CookieJar.h"
> #include "DocumentLoader.h"
> #include "Frame.h"
>+#include "FrameLoader.h"
>+#include "FrameLoaderClient.h"
> #include "FrameView.h"
> #include "GraphicsContext.h"
> #include "KURL.h"
>@@ -268,7 +270,7 @@ void MediaPlayerPrivateQuickTimeVisualCo
> 
>     KURL movieURL = KURL(KURL(), url);
>     Vector<Cookie> documentCookies;
>-    if (!getRawCookies(frame->document(), movieURL, documentCookies))
>+    if (!frame->loader()->client()->cookieJar().getRawCookies(movieURL, document->firstPartyForCookies(), documentCookies))
>         return;
> 
>     for (size_t ndx = 0; ndx < documentCookies.size(); ndx++) {
>Index: Source/WebCore/platform/mac/CookieJar.mm
>===================================================================
>--- Source/WebCore/platform/mac/CookieJar.mm	(revision 132793)
>+++ Source/WebCore/platform/mac/CookieJar.mm	(working copy)
>@@ -76,7 +76,7 @@ static RetainPtr<NSArray> filterCookies(
>     return filteredCookies;
> }
> 
>-String cookies(const Document*, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
> 
>@@ -95,7 +95,7 @@ String cookies(const Document*, const KU
>     return String();
> }
> 
>-String cookieRequestHeaderFieldValue(const Document*, const KURL& url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
> 
>@@ -114,7 +114,7 @@ String cookieRequestHeaderFieldValue(con
>     return String();
> }
> 
>-void setCookies(Document* document, const KURL& url, const String& cookieStr)
>+void CookieJar::setCookies(const KURL& url, const KURL& firstPartyForCookies, const String& cookieStr)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
> 
>@@ -132,15 +132,15 @@ void setCookies(Document* document, cons
> 
> #if USE(CFURLSTORAGESESSIONS)
>     if (RetainPtr<CFHTTPCookieStorageRef> cfCookieStorage = currentCFHTTPCookieStorage())
>-        wkSetHTTPCookiesForURL(cfCookieStorage.get(), filteredCookies.get(), cookieURL, document->firstPartyForCookies());
>+        wkSetHTTPCookiesForURL(cfCookieStorage.get(), filteredCookies.get(), cookieURL, firstPartyForCookies);
>     else
> #endif
>-        [[NSHTTPCookieStorage sharedHTTPCookieStorage] setCookies:filteredCookies.get() forURL:cookieURL mainDocumentURL:document->firstPartyForCookies()];
>+        [[NSHTTPCookieStorage sharedHTTPCookieStorage] setCookies:filteredCookies.get() forURL:cookieURL mainDocumentURL:firstPartyForCookies];
> 
>     END_BLOCK_OBJC_EXCEPTIONS;
> }
> 
>-bool cookiesEnabled(const Document*)
>+bool CookieJar::cookiesEnabled(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
> 
>@@ -158,7 +158,7 @@ bool cookiesEnabled(const Document*)
>     return false;
> }
> 
>-bool getRawCookies(const Document*, const KURL& url, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& url, const KURL& /*firstPartyForCookies*/, Vector<Cookie>& rawCookies)
> {
>     rawCookies.clear();
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
>@@ -191,7 +191,7 @@ bool getRawCookies(const Document*, cons
>     return true;
> }
> 
>-void deleteCookie(const Document*, const KURL& url, const String& cookieName)
>+void CookieJar::deleteCookie(const KURL& url, const String& cookieName)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
> 
>@@ -225,7 +225,7 @@ void deleteCookie(const Document*, const
>     END_BLOCK_OBJC_EXCEPTIONS;
> }
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& hostnames)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
>     
>@@ -238,7 +238,7 @@ void getHostnamesWithCookies(HashSet<Str
>     END_BLOCK_OBJC_EXCEPTIONS;
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& hostname)
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
>     
>@@ -255,7 +255,7 @@ void deleteCookiesForHostname(const Stri
>     END_BLOCK_OBJC_EXCEPTIONS;
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     BEGIN_BLOCK_OBJC_EXCEPTIONS;
>     
>Index: Source/WebCore/platform/network/cf/CookieJarCFNet.cpp
>===================================================================
>--- Source/WebCore/platform/network/cf/CookieJarCFNet.cpp	(revision 132793)
>+++ Source/WebCore/platform/network/cf/CookieJarCFNet.cpp	(working copy)
>@@ -30,7 +30,6 @@
> 
> #include "Cookie.h"
> #include "CookieStorageCFNet.h"
>-#include "Document.h"
> #include "KURL.h"
> #include "ResourceHandle.h"
> #include "SoftLinking.h"
>@@ -95,7 +94,7 @@ static RetainPtr<CFArrayRef> filterCooki
>     return filteredCookies;
> }
> 
>-void setCookies(Document* document, const KURL& url, const String& value)
>+void CookieJar::setCookies(const KURL& url, const KURL& firstPartyForCookies, const String& value)
> {
>     // <rdar://problem/5632883> CFHTTPCookieStorage stores an empty cookie, which would be sent as "Cookie: =".
>     if (value.isEmpty())
>@@ -106,7 +105,7 @@ void setCookies(Document* document, cons
>         return;
> 
>     RetainPtr<CFURLRef> urlCF(AdoptCF, url.createCFURL());
>-    RetainPtr<CFURLRef> firstPartyForCookiesCF(AdoptCF, document->firstPartyForCookies().createCFURL());
>+    RetainPtr<CFURLRef> firstPartyForCookiesCF(AdoptCF, firstPartyForCookies.createCFURL());
> 
>     // <http://bugs.webkit.org/show_bug.cgi?id=6531>, <rdar://4409034>
>     // cookiesWithResponseHeaderFields doesn't parse cookies without a value
>@@ -123,7 +122,7 @@ void setCookies(Document* document, cons
>     CFHTTPCookieStorageSetCookies(cookieStorage.get(), filterCookies(cookiesCF.get()).get(), urlCF.get(), firstPartyForCookiesCF.get());
> }
> 
>-String cookies(const Document* /*document*/, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>@@ -137,7 +136,7 @@ String cookies(const Document* /*documen
>     return (CFStringRef)CFDictionaryGetValue(headerCF.get(), s_cookieCF);
> }
> 
>-String cookieRequestHeaderFieldValue(const Document* /*document*/, const KURL& url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>@@ -151,7 +150,7 @@ String cookieRequestHeaderFieldValue(con
>     return (CFStringRef)CFDictionaryGetValue(headerCF.get(), s_cookieCF);
> }
> 
>-bool cookiesEnabled(const Document* /*document*/)
>+bool CookieJar::cookiesEnabled(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/)
> {
>     CFHTTPCookieStorageAcceptPolicy policy = CFHTTPCookieStorageAcceptPolicyOnlyFromMainDocumentDomain;
>     if (RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage())
>@@ -159,7 +158,7 @@ bool cookiesEnabled(const Document* /*do
>     return policy == CFHTTPCookieStorageAcceptPolicyOnlyFromMainDocumentDomain || policy == CFHTTPCookieStorageAcceptPolicyAlways;
> }
> 
>-bool getRawCookies(const Document*, const KURL& url, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& url, const KURL& /*firstPartyForCookies*/, Vector<Cookie>& rawCookies)
> {
>     rawCookies.clear();
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>@@ -193,7 +192,7 @@ bool getRawCookies(const Document*, cons
>     return true;
> }
> 
>-void deleteCookie(const Document*, const KURL& url, const String& name)
>+void CookieJar::deleteCookie(const KURL& url, const String& name)
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>@@ -214,7 +213,7 @@ void deleteCookie(const Document*, const
>     }
> }
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& hostnames)
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>@@ -232,7 +231,7 @@ void getHostnamesWithCookies(HashSet<Str
>     }
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& hostname)
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>@@ -251,7 +250,7 @@ void deleteCookiesForHostname(const Stri
>     }
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     RetainPtr<CFHTTPCookieStorageRef> cookieStorage = currentCFHTTPCookieStorage();
>     if (!cookieStorage)
>Index: Source/WebCore/platform/network/chromium/CookieJarChromium.cpp
>===================================================================
>--- Source/WebCore/platform/network/chromium/CookieJarChromium.cpp	(revision 132793)
>+++ Source/WebCore/platform/network/chromium/CookieJarChromium.cpp	(working copy)
>@@ -32,52 +32,74 @@
> #include "CookieJar.h"
> 
> #include "Cookie.h"
>-#include "Document.h"
>-#include "PlatformSupport.h"
>+#include <public/WebCookie.h>
>+#include <public/WebCookieJar.h>
>+#include <public/WebString.h>
>+#include <public/WebURL.h>
>+#include <public/WebVector.h>
> 
> namespace WebCore {
> 
>-void setCookies(Document* document, const KURL& url, const String& value)
>+void CookieJar::setCookies(const KURL& url, const KURL& firstPartyForCookies, const String& value)
> {
>-    PlatformSupport::setCookies(document, url, value);
>+    if (m_webCookieJar)
>+        m_webCookieJar->setCookie(url, firstPartyForCookies, value);
> }
> 
>-String cookies(const Document* document, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& firstPartyForCookies)
> {
>-    return PlatformSupport::cookies(document, url);
>+    if (!m_webCookieJar)
>+        return String();
>+    return m_webCookieJar->cookies(url, firstPartyForCookies);
> }
> 
>-String cookieRequestHeaderFieldValue(const Document* document, const KURL& url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& firstPartyForCookies)
> {
>-    return PlatformSupport::cookieRequestHeaderFieldValue(document, url);
>+    if (!m_webCookieJar)
>+        return String();
>+    return m_webCookieJar->cookieRequestHeaderFieldValue(url, firstPartyForCookies);
> }
> 
>-bool cookiesEnabled(const Document* document)
>+bool CookieJar::cookiesEnabled(const KURL& url, const KURL& firstPartyForCookies)
> {
>-    return PlatformSupport::cookiesEnabled(document);
>+    if (!m_webCookieJar)
>+        return false;
>+    return m_webCookieJar->cookiesEnabled(url, firstPartyForCookies);
> }
> 
>-bool getRawCookies(const Document* document, const KURL& url, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& url, const KURL& firstPartyForCookies, Vector<Cookie>& rawCookies)
> {
>-    return PlatformSupport::rawCookies(document, url, rawCookies);
>+    rawCookies.clear();
>+    WebKit::WebVector<WebKit::WebCookie> webCookies;
>+
>+    if (m_webCookieJar)
>+        m_webCookieJar->rawCookies(url, firstPartyForCookies, webCookies);
>+
>+    for (unsigned i = 0; i < webCookies.size(); ++i) {
>+        const WebKit::WebCookie& webCookie = webCookies[i];
>+        Cookie cookie(webCookie.name, webCookie.value, webCookie.domain, webCookie.path, webCookie.expires, webCookie.httpOnly, webCookie.secure, webCookie.session);
>+        rawCookies.append(cookie);
>+    }
>+    return true;
> }
> 
>-void deleteCookie(const Document* document, const KURL& url, const String& cookieName)
>+void CookieJar::deleteCookie(const KURL& url, const String& cookieName)
> {
>-    return PlatformSupport::deleteCookie(document, url, cookieName);
>+    if (m_webCookieJar)
>+        m_webCookieJar->deleteCookie(url, cookieName);
> }
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& /*hostnames*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& /*hostname*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     // FIXME: Not yet implemented
> }
>Index: Source/WebCore/platform/network/curl/CookieJarCurl.cpp
>===================================================================
>--- Source/WebCore/platform/network/curl/CookieJarCurl.cpp	(revision 132793)
>+++ Source/WebCore/platform/network/curl/CookieJarCurl.cpp	(working copy)
>@@ -18,7 +18,6 @@
> #include "CookieJar.h"
> 
> #include "Cookie.h"
>-#include "Document.h"
> #include "KURL.h"
> #include <wtf/HashMap.h>
> #include <wtf/text/StringHash.h>
>@@ -26,37 +25,37 @@
> 
> namespace WebCore {
> 
>-static HashMap<String, String> cookieJar;
>+static HashMap<String, String> s_cookieJar;
> 
>-void setCookies(Document* /*document*/, const KURL& url, const String& value)
>+void CookieJar::setCookies(const KURL& url, const KURL& /*firstPartyForCookies*/, const String& value)
> {
>-    cookieJar.set(url.string(), value);
>+    s_cookieJar.set(url.string(), value);
> }
> 
>-String cookies(const Document* /*document*/, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>-    return cookieJar.get(url.string());
>+    return s_cookieJar.get(url.string());
> }
> 
>-String cookieRequestHeaderFieldValue(const Document* /*document*/, const KURL& url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     // FIXME: include HttpOnly cookie.
>-    return cookieJar.get(url.string());
>+    return s_cookieJar.get(url.string());
> }
> 
>-bool cookiesEnabled(const Document* /*document*/)
>+bool CookieJar::cookiesEnabled(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/)
> {
>     return true;
> }
> 
>-bool getRawCookies(const Document*, const KURL&, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/, Vector<Cookie>& rawCookies)
> {
>     // FIXME: Not yet implemented
>     rawCookies.clear();
>     return false; // return true when implemented
> }
> 
>-void deleteCookie(const Document*, const KURL&, const String&)
>+void CookieJar::deleteCookie(const KURL& /*url*/, const String& /*cookieName*/)
> {
>     // FIXME: Not yet implemented
> }
>@@ -68,17 +67,17 @@ void setCookieStoragePrivateBrowsingEnab
> }
> #endif
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& /*hostnames*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& /*hostname*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     // FIXME: Not yet implemented
> }
>Index: Source/WebCore/platform/network/win/CookieJarWin.cpp
>===================================================================
>--- Source/WebCore/platform/network/win/CookieJarWin.cpp	(revision 132793)
>+++ Source/WebCore/platform/network/win/CookieJarWin.cpp	(working copy)
>@@ -28,7 +28,6 @@
> 
> #include "Cookie.h"
> #include "KURL.h"
>-#include "Document.h"
> #include "ResourceHandle.h"
> #include <windows.h>
> #include <Wininet.h>
>@@ -36,15 +35,16 @@
> 
> namespace WebCore {
> 
>-void setCookies(Document* /*document*/, const KURL& url, const String& value)
>+
>+void CookieJar::setCookies(const KURL& url, const KURL& /*firstPartyForCookies*/, const String& value)
> {
>-    // FIXME: Deal with document->firstPartyForCookies().
>+    // FIXME: Deal with firstPartyForCookies.
>     String str = url.string();
>     String val = value;
>     InternetSetCookie(str.charactersWithNullTermination(), 0, val.charactersWithNullTermination());
> }
> 
>-String cookies(const Document* /*document*/, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>     String str = url.string();
> 
>@@ -63,40 +63,40 @@ String cookies(const Document* /*documen
>     return String::adopt(buffer);
> }
> 
>-String cookieRequestHeaderFieldValue(const Document* document, const KURL& url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& firstPartyForCookies)
> {
>     // FIXME: include HttpOnly cookie
>-    return cookies(document, url);
>+    return cookies(url, firstPartyForCookies);
> }
> 
>-bool cookiesEnabled(const Document* /*document*/)
>+bool CookieJar::cookiesEnabled(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/)
> {
>     return true;
> }
> 
>-bool getRawCookies(const Document*, const KURL&, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/, Vector<Cookie>& rawCookies)
> {
>     // FIXME: Not yet implemented
>     rawCookies.clear();
>     return false; // return true when implemented
> }
> 
>-void deleteCookie(const Document*, const KURL&, const String&)
>+void CookieJar::deleteCookie(const KURL& /*url*/, const String& /*cookieName*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& /*hostnames*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& /*hostname*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     // FIXME: Not yet implemented
> }
>Index: Source/WebCore/platform/qt/CookieJarQt.cpp
>===================================================================
>--- Source/WebCore/platform/qt/CookieJarQt.cpp	(revision 132793)
>+++ Source/WebCore/platform/qt/CookieJarQt.cpp	(working copy)
>@@ -31,9 +31,6 @@
> 
> #include "Cookie.h"
> #include "CookieJar.h"
>-#include "Document.h"
>-#include "Frame.h"
>-#include "FrameLoader.h"
> #include "KURL.h"
> #include "NetworkingContext.h"
> #include "ThirdPartyCookiesQt.h"
>@@ -49,31 +46,17 @@ namespace WebCore {
> 
> static SharedCookieJarQt* s_sharedCookieJarQt = 0;
> 
>-static NetworkingContext* networkingContext(const Document* document)
>+void CookieJar::setCookies(const KURL& url, const KURL& firstPartyForCookies, const String& value)
> {
>-    if (!document)
>-        return 0;
>-    Frame* frame = document->frame();
>-    if (!frame)
>-        return 0;
>-    FrameLoader* loader = frame->loader();
>-    if (!loader)
>-        return 0;
>-    return loader->networkingContext();
>-}
>-
>-void setCookies(Document* document, const KURL& url, const String& value)
>-{
>-    NetworkingContext* context = networkingContext(document);
>-    if (!context)
>+    if (!m_networkingContext)
>         return;
>-    QNetworkCookieJar* jar = context->networkAccessManager()->cookieJar();
>+    QNetworkCookieJar* jar = m_networkingContext->networkAccessManager()->cookieJar();
>     if (!jar)
>         return;
> 
>     QUrl urlForCookies(url);
>-    QUrl firstPartyUrl(document->firstPartyForCookies());
>-    if (!thirdPartyCookiePolicyPermits(context, urlForCookies, firstPartyUrl))
>+    QUrl firstPartyUrl(firstPartyForCookies());
>+    if (!thirdPartyCookiePolicyPermits(m_networkingContext, urlForCookies, firstPartyUrl))
>         return;
> 
>     QList<QNetworkCookie> cookies = QNetworkCookie::parseCookies(QString(value).toLatin1());
>@@ -88,16 +71,15 @@ void setCookies(Document* document, cons
>     jar->setCookiesFromUrl(cookies, urlForCookies);
> }
> 
>-String cookies(const Document* document, const KURL& url)
>+String CookieJar::cookies(const KURL& url, const KURL& firstPartyForCookies)
> {
>-    NetworkingContext* context = networkingContext(document);
>-    if (!context)
>+    if (!m_networkingContext)
>         return String();
>-    QNetworkCookieJar* jar = context->networkAccessManager()->cookieJar();
>+    QNetworkCookieJar* jar = m_networkingContext->networkAccessManager()->cookieJar();
> 
>     QUrl urlForCookies(url);
>-    QUrl firstPartyUrl(document->firstPartyForCookies());
>-    if (!thirdPartyCookiePolicyPermits(context, urlForCookies, firstPartyUrl))
>+    QUrl firstPartyUrl(firstPartyForCookies());
>+    if (!thirdPartyCookiePolicyPermits(m_networkingContext, urlForCookies, firstPartyUrl))
>         return String();
> 
>     QList<QNetworkCookie> cookies = jar->cookiesForUrl(urlForCookies);
>@@ -114,12 +96,11 @@ String cookies(const Document* document,
>     return resultCookies.join(QLatin1String("; "));
> }
> 
>-String cookieRequestHeaderFieldValue(const Document* document, const KURL &url)
>+String CookieJar::cookieRequestHeaderFieldValue(const KURL& url, const KURL& /*firstPartyForCookies*/)
> {
>-    NetworkingContext* context = networkingContext(document);
>-    if (!context)
>+    if (!m_networkingContext)
>         return String();
>-    QNetworkCookieJar* jar = context->networkAccessManager()->cookieJar();
>+    QNetworkCookieJar* jar = m_networkingContext->networkAccessManager()->cookieJar();
> 
>     QList<QNetworkCookie> cookies = jar->cookiesForUrl(QUrl(url));
>     if (cookies.isEmpty())
>@@ -132,41 +113,40 @@ String cookieRequestHeaderFieldValue(con
>     return resultCookies.join(QLatin1String("; "));
> }
> 
>-bool cookiesEnabled(const Document* document)
>+bool CookieJar::cookiesEnabled(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/)
> {
>-    NetworkingContext* context = networkingContext(document);
>-    if (!context)
>+    if (!m_networkingContext)
>         return false;
>-    return context->networkAccessManager()->cookieJar();
>+    return m_networkingContext->networkAccessManager()->cookieJar();
> }
> 
>-bool getRawCookies(const Document*, const KURL&, Vector<Cookie>& rawCookies)
>+bool CookieJar::getRawCookies(const KURL& /*url*/, const KURL& /*firstPartyForCookies*/, Vector<Cookie>& rawCookies)
> {
>     // FIXME: Not yet implemented
>     rawCookies.clear();
>     return false; // return true when implemented
> }
> 
>-void deleteCookie(const Document*, const KURL&, const String&)
>+void CookieJar::deleteCookie(const KURL& /*url*/, const String& /*cookieName*/)
> {
>     // FIXME: Not yet implemented
> }
> 
>-void getHostnamesWithCookies(HashSet<String>& hostnames)
>+void CookieJar::getHostnamesWithCookies(HashSet<String>& hostnames)
> {
>     SharedCookieJarQt* jar = SharedCookieJarQt::shared();
>     if (jar)
>         jar->getHostnamesWithCookies(hostnames);
> }
> 
>-void deleteCookiesForHostname(const String& hostname)
>+void CookieJar::deleteCookiesForHostname(const String& hostname)
> {
>     SharedCookieJarQt* jar = SharedCookieJarQt::shared();
>     if (jar)
>         jar->deleteCookiesForHostname(hostname);
> }
> 
>-void deleteAllCookies()
>+void CookieJar::deleteAllCookies()
> {
>     SharedCookieJarQt* jar = SharedCookieJarQt::shared();
>     if (jar)
>Index: Source/WebCore/plugins/PluginView.cpp
>===================================================================
>--- Source/WebCore/plugins/PluginView.cpp	(revision 132793)
>+++ Source/WebCore/plugins/PluginView.cpp	(working copy)
>@@ -1392,7 +1392,7 @@ NPError PluginView::getValueForURL(NPNUR
>         if (u.isValid()) {
>             Frame* frame = getFrame(parentFrame(), m_element);
>             if (frame) {
>-                const CString cookieStr = cookies(frame->document(), u).utf8();
>+                const CString cookieStr = frame->loader()->client()->cookieJar().cookies(u, frame->document()->firstPartyForCookies()).utf8();
>                 if (!cookieStr.isNull()) {
>                     const int size = cookieStr.length();
>                     *value = static_cast<char*>(NPN_MemAlloc(size+1));
>@@ -1454,7 +1454,7 @@ NPError PluginView::setValueForURL(NPNUR
>             const String cookieStr = String::fromUTF8(value, len);
>             Frame* frame = getFrame(parentFrame(), m_element);
>             if (frame && !cookieStr.isEmpty())
>-                setCookies(frame->document(), u, cookieStr);
>+                frame->loader()->client()->cookieJar().setCookies(u, frame->document()->firstPartyForCookies(), cookieStr);
>         } else
>             result = NPERR_INVALID_URL;
>         break;
>Index: Source/WebKit/chromium/ChangeLog
>===================================================================
>--- Source/WebKit/chromium/ChangeLog	(revision 132828)
>+++ Source/WebKit/chromium/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2012-10-29  Mark Pilgrim  <pilgrim@chromium.org>
>+
>+        [Chromium] Remove cookie-related functions from PlatformSupport
>+        https://bugs.webkit.org/show_bug.cgi?id=99340
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Additional information of the change such as approach, rationale. Please add per-function descriptions below (OOPS!).
>+
>+        * src/FrameLoaderClientImpl.cpp:
>+        (WebKit::FrameLoaderClientImpl::cookieJar):
>+        * src/FrameLoaderClientImpl.h:
>+        (FrameLoaderClientImpl):
>+        * src/PlatformSupport.cpp:
>+        (WebCore):
>+
> 2012-10-29  Patrick Dubroy  <dubroy@chromium.org>
> 
>         Web Inspector: Style toolbar to match Chromium toolbar on Chromium/Mac.
>Index: Source/WebKit/chromium/src/FrameLoaderClientImpl.cpp
>===================================================================
>--- Source/WebKit/chromium/src/FrameLoaderClientImpl.cpp	(revision 132793)
>+++ Source/WebKit/chromium/src/FrameLoaderClientImpl.cpp	(working copy)
>@@ -1651,5 +1651,14 @@ void FrameLoaderClientImpl::dispatchWill
> }
> #endif
> 
>+WebCore::CookieJar FrameLoaderClientImpl::cookieJar()
>+{
>+    if (!m_webFrame || !m_webFrame->client())
>+        return WebCore::CookieJar(0);
>+    WebCookieJar* webCookieJar = m_webFrame->client()->cookieJar(m_webFrame);
>+    if (!webCookieJar)
>+        webCookieJar = WebKit::Platform::current()->cookieJar();
>+    return WebCore::CookieJar(webCookieJar);
>+}
> 
> } // namespace WebKit
>Index: Source/WebKit/chromium/src/FrameLoaderClientImpl.h
>===================================================================
>--- Source/WebKit/chromium/src/FrameLoaderClientImpl.h	(revision 132793)
>+++ Source/WebKit/chromium/src/FrameLoaderClientImpl.h	(working copy)
>@@ -225,6 +225,8 @@ public:
>     virtual void dispatchWillStartUsingPeerConnectionHandler(WebCore::RTCPeerConnectionHandler*) OVERRIDE;
> #endif
> 
>+    virtual WebCore::CookieJar cookieJar() OVERRIDE;
>+
> private:
>     void makeDocumentView();
> 
>Index: Source/WebKit/chromium/src/PlatformSupport.cpp
>===================================================================
>--- Source/WebKit/chromium/src/PlatformSupport.cpp	(revision 132793)
>+++ Source/WebKit/chromium/src/PlatformSupport.cpp	(working copy)
>@@ -100,86 +100,6 @@ using namespace WebKit;
> 
> namespace WebCore {
> 
>-static WebCookieJar* getCookieJar(const Document* document)
>-{
>-    WebFrameImpl* frameImpl = WebFrameImpl::fromFrame(document->frame());
>-    if (!frameImpl || !frameImpl->client())
>-        return 0;
>-    WebCookieJar* cookieJar = frameImpl->client()->cookieJar(frameImpl);
>-    if (!cookieJar)
>-        cookieJar = WebKit::Platform::current()->cookieJar();
>-    return cookieJar;
>-}
>-
>-// Cookies --------------------------------------------------------------------
>-
>-void PlatformSupport::setCookies(const Document* document, const KURL& url,
>-                                const String& value)
>-{
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        cookieJar->setCookie(url, document->firstPartyForCookies(), value);
>-}
>-
>-String PlatformSupport::cookies(const Document* document, const KURL& url)
>-{
>-    String result;
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        result = cookieJar->cookies(url, document->firstPartyForCookies());
>-    return result;
>-}
>-
>-String PlatformSupport::cookieRequestHeaderFieldValue(const Document* document,
>-                                                     const KURL& url)
>-{
>-    String result;
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        result = cookieJar->cookieRequestHeaderFieldValue(url, document->firstPartyForCookies());
>-    return result;
>-}
>-
>-bool PlatformSupport::rawCookies(const Document* document, const KURL& url, Vector<Cookie>& rawCookies)
>-{
>-    rawCookies.clear();
>-    WebVector<WebCookie> webCookies;
>-
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        cookieJar->rawCookies(url, document->firstPartyForCookies(), webCookies);
>-
>-    for (unsigned i = 0; i < webCookies.size(); ++i) {
>-        const WebCookie& webCookie = webCookies[i];
>-        Cookie cookie(webCookie.name,
>-                      webCookie.value,
>-                      webCookie.domain,
>-                      webCookie.path,
>-                      webCookie.expires,
>-                      webCookie.httpOnly,
>-                      webCookie.secure,
>-                      webCookie.session);
>-        rawCookies.append(cookie);
>-    }
>-    return true;
>-}
>-
>-void PlatformSupport::deleteCookie(const Document* document, const KURL& url, const String& cookieName)
>-{
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        cookieJar->deleteCookie(url, cookieName);
>-}
>-
>-bool PlatformSupport::cookiesEnabled(const Document* document)
>-{
>-    bool result = false;
>-    WebCookieJar* cookieJar = getCookieJar(document);
>-    if (cookieJar)
>-        result = cookieJar->cookiesEnabled(document->cookieURL(), document->firstPartyForCookies());
>-    return result;
>-}
>-
> // Font -----------------------------------------------------------------------
> 
> #if OS(WINDOWS)
>Index: Source/WebKit/qt/ChangeLog
>===================================================================
>--- Source/WebKit/qt/ChangeLog	(revision 132828)
>+++ Source/WebKit/qt/ChangeLog	(working copy)
>@@ -1,3 +1,19 @@
>+2012-10-29  Mark Pilgrim  <pilgrim@chromium.org>
>+
>+        [Chromium] Remove cookie-related functions from PlatformSupport
>+        https://bugs.webkit.org/show_bug.cgi?id=99340
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Additional information of the change such as approach, rationale. Please add per-function descriptions below (OOPS!).
>+
>+        * WebCoreSupport/FrameLoaderClientQt.cpp:
>+        (WebCore::FrameLoaderClientQt::cookieJar):
>+        (WebCore):
>+        * WebCoreSupport/FrameLoaderClientQt.h:
>+        (WebCore):
>+        (FrameLoaderClientQt):
>+
> 2012-10-24  Brady Eidson  <beidson@apple.com>
> 
>         Add a strategy for loader customization.
>Index: Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp
>===================================================================
>--- Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp	(revision 132793)
>+++ Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp	(working copy)
>@@ -1676,6 +1676,13 @@ void FrameLoaderClientQt::emitLoadFinish
>     emit m_webFrame->loadFinished(ok);
> }
> 
>+WebCore::CookieJar FrameLoaderClientQt::cookieJar()
>+{
>+    if (!m_frame || !m_frame->loader())
>+        return WebCore::CookieJar(0);
>+    return WebCore::CookieJar(m_frame->loader()->networkingContext());
>+}
>+
> }
> 
> #include "moc_FrameLoaderClientQt.cpp"
>Index: Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.h
>===================================================================
>--- Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.h	(revision 132793)
>+++ Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.h	(working copy)
>@@ -31,6 +31,7 @@
> #define FrameLoaderClientQt_h
> 
> 
>+#include "CookieJar.h"
> #include "Frame.h"
> #include "FrameLoader.h"
> #include "FrameLoaderClient.h"
>@@ -53,6 +54,7 @@ class QWebFrame;
> namespace WebCore {
> 
> class AuthenticationChallenge;
>+class CookieJar;
> class DocumentLoader;
> class Element;
> class FormState;
>@@ -227,6 +229,8 @@ public:
> 
>     const KURL& lastRequestedUrl() const { return m_lastRequestedUrl; }
> 
>+    virtual WebCore::CookieJar cookieJar() OVERRIDE;
>+
>     static bool dumpFrameLoaderCallbacks;
>     static bool dumpProgressFinishedCallback;
>     static bool dumpUserGestureInFrameLoaderCallbacks;

Actions: View | Formatted Diff | Diff

Attachments on bug 99340: 168741 | 169674 | 169683 | 170450 | 170456 | 170459 | 171274 | 171280 | 171282 | 171284 | 171287 | 171290 | 171293 | 171296 | 171684 | 171848 | 171875 | 171908 | 172373 | 172384 | 172862 | 172878 | 173031 | 174188 | 174206 | 174675 | 174753