Lars Knoll <lars@trolltech.com>
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Jan 2008 10:03:40 +0000 (10:03 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Jan 2008 10:03:40 +0000 (10:03 +0000)
cleanup the CookieJarQt implementation and implement cookiesEnabled.

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

WebCore/ChangeLog
WebCore/platform/qt/CookieJarQt.cpp

index 8c67d37ecbdac69be742e6a671547978fc72b203..b6ef1c78583d1417ae0369f02d84be916dc71719 100644 (file)
@@ -1,3 +1,15 @@
+2008-01-22  Lars Knoll  <lars@trolltech.com>
+
+        Reviewed by Simon.
+
+        cleanup the CookieJarQt implementation and implement cookiesEnabled.
+
+        * platform/qt/CookieJarQt.cpp:
+        (WebCore::cookieJar):
+        (WebCore::setCookies):
+        (WebCore::cookies):
+        (WebCore::cookiesEnabled):
+
 2008-01-22  Lars Knoll  <lars@trolltech.com>
 
         Reviewed by Simon.
 2008-01-22  Lars Knoll  <lars@trolltech.com>
 
         Reviewed by Simon.
index 1759a0bae205e6e3053dda5a65a85dc9dae3f533..4138ab7ee339a51e3bb4a123878dec9ee1fbdf4e 100644 (file)
 
 namespace WebCore {
 
 
 namespace WebCore {
 
-void setCookies(Document* document, const KURL& url, const KURL& policyURL, const String& value)
-{
-    QUrl u(url);
-    QUrl p(policyURL);
 #if QT_VERSION >= 0x040400
 #if QT_VERSION >= 0x040400
+static QNetworkCookieJar *cookieJar(const Document *document)
+{
     Frame *frame = document->frame();
     if (!frame)
     Frame *frame = document->frame();
     if (!frame)
-        return;
+        return 0;
     FrameLoader *loader = frame->loader();
     if (!loader)
     FrameLoader *loader = frame->loader();
     if (!loader)
-        return;
+        return 0;
     QWebFrame* webFrame = static_cast<FrameLoaderClientQt*>(loader->client())->webFrame();
     QWebPage* page = webFrame->page();
     QNetworkAccessManager* manager = page->networkAccessManager();
     QNetworkCookieJar* jar = manager->cookieJar();
     QWebFrame* webFrame = static_cast<FrameLoaderClientQt*>(loader->client())->webFrame();
     QWebPage* page = webFrame->page();
     QNetworkAccessManager* manager = page->networkAccessManager();
     QNetworkCookieJar* jar = manager->cookieJar();
+    return jar;
+}
+#endif
+
+void setCookies(Document* document, const KURL& url, const KURL& policyURL, const String& value)
+{
+    QUrl u(url);
+    QUrl p(policyURL);
+#if QT_VERSION >= 0x040400
+    QNetworkCookieJar* jar = cookieJar(document);
+    if (!jar)
+        return;
 
     QList<QNetworkCookie> cookies = QNetworkCookie::parseCookies(QString(value).toAscii());
     jar->setCookiesFromUrl(cookies, p);
 
     QList<QNetworkCookie> cookies = QNetworkCookie::parseCookies(QString(value).toAscii());
     jar->setCookiesFromUrl(cookies, p);
@@ -72,16 +82,9 @@ String cookies(const Document* document, const KURL& url)
 {
     QUrl u(url);
 #if QT_VERSION >= 0x040400
 {
     QUrl u(url);
 #if QT_VERSION >= 0x040400
-    Frame *frame = document->frame();
-    if (!frame)
-        return String();
-    FrameLoader *loader = frame->loader();
-    if (!loader)
+    QNetworkCookieJar* jar = cookieJar(document);
+    if (!jar)
         return String();
         return String();
-    QWebFrame* webFrame = static_cast<FrameLoaderClientQt*>(loader->client())->webFrame();
-    QWebPage* page = webFrame->page();
-    QNetworkAccessManager* manager = page->networkAccessManager();
-    QNetworkCookieJar* jar = manager->cookieJar();
 
     QList<QNetworkCookie> cookies = jar->cookiesForUrl(u);
     if (cookies.isEmpty())
 
     QList<QNetworkCookie> cookies = jar->cookiesForUrl(u);
     if (cookies.isEmpty())
@@ -104,8 +107,8 @@ String cookies(const Document* document, const KURL& url)
 bool cookiesEnabled(const Document* document)
 {
 #if QT_VERSION >= 0x040400
 bool cookiesEnabled(const Document* document)
 {
 #if QT_VERSION >= 0x040400
-    // ###
-    return true;
+    QNetworkCookieJar* jar = cookieJar(document);
+    return (jar != 0);
 #else
     return QCookieJar::cookieJar()->isEnabled();
 #endif
 #else
     return QCookieJar::cookieJar()->isEnabled();
 #endif