Title: [136350] trunk/Source/WebCore
Revision
136350
Author
[email protected]
Date
2012-12-02 12:29:13 -0800 (Sun, 02 Dec 2012)

Log Message

[BlackBerry] Adding checks for null when checking cookie status in CookieJarBlackBerry
https://bugs.webkit.org/show_bug.cgi?id=103748

Patch by Otto Derek Cheung <[email protected]> on 2012-12-02
Reviewed by Rob Buis.

The logic is flipped in the previous patch.

* loader/blackberry/CookieJarBlackBerry.cpp:
(WebCore::cookiesEnabled):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (136349 => 136350)


--- trunk/Source/WebCore/ChangeLog	2012-12-02 19:56:07 UTC (rev 136349)
+++ trunk/Source/WebCore/ChangeLog	2012-12-02 20:29:13 UTC (rev 136350)
@@ -1,3 +1,15 @@
+2012-12-02  Otto Derek Cheung  <[email protected]>
+
+        [BlackBerry] Adding checks for null when checking cookie status in CookieJarBlackBerry
+        https://bugs.webkit.org/show_bug.cgi?id=103748
+
+        Reviewed by Rob Buis.
+
+        The logic is flipped in the previous patch.
+
+        * loader/blackberry/CookieJarBlackBerry.cpp:
+        (WebCore::cookiesEnabled):
+
 2012-12-02  Robert Hogan  <[email protected]>
 
         Bad rendering of web page because of image's height is set to 100%

Modified: trunk/Source/WebCore/loader/blackberry/CookieJarBlackBerry.cpp (136349 => 136350)


--- trunk/Source/WebCore/loader/blackberry/CookieJarBlackBerry.cpp	2012-12-02 19:56:07 UTC (rev 136349)
+++ trunk/Source/WebCore/loader/blackberry/CookieJarBlackBerry.cpp	2012-12-02 20:29:13 UTC (rev 136350)
@@ -52,9 +52,9 @@
         cookieManager().setCookies(url, value, NoHttpOnlyCookie);
 }
 
-bool cookiesEnabled(Document const* document)
+bool cookiesEnabled(const Document* document)
 {
-    return !(document && document->settings() && document->settings()->cookieEnabled());
+    return document && document->settings() && document->settings()->cookieEnabled();
 }
 
 bool getRawCookies(const Document* document, const KURL& url, Vector<Cookie>& rawCookies)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to