Title: [138568] trunk/Source/WebCore
Revision
138568
Author
[email protected]
Date
2012-12-29 11:10:56 -0800 (Sat, 29 Dec 2012)

Log Message

[BlackBerry] Cookies with an IP domain are not being loaded properly into memory
https://bugs.webkit.org/show_bug.cgi?id=105859

PR 271597

Patch by Otto Derek Cheung <[email protected]> on 2012-12-29
Reviewed by George Staikos.

The isIP flag isn't persistented in the database. We need to check it
again when we load it into memory.

* platform/blackberry/CookieManager.cpp:
(WebCore::CookieManager::getBackingStoreCookies):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (138567 => 138568)


--- trunk/Source/WebCore/ChangeLog	2012-12-29 18:11:57 UTC (rev 138567)
+++ trunk/Source/WebCore/ChangeLog	2012-12-29 19:10:56 UTC (rev 138568)
@@ -1,3 +1,18 @@
+2012-12-29  Otto Derek Cheung  <[email protected]>
+
+        [BlackBerry] Cookies with an IP domain are not being loaded properly into memory
+        https://bugs.webkit.org/show_bug.cgi?id=105859
+
+        PR 271597
+
+        Reviewed by George Staikos.
+
+        The isIP flag isn't persistented in the database. We need to check it
+        again when we load it into memory.
+
+        * platform/blackberry/CookieManager.cpp:
+        (WebCore::CookieManager::getBackingStoreCookies):
+
 2012-12-29  Kondapally Kalyan  <[email protected]>
 
         [EFL][WebGL] Refactor GLXImplementation.

Modified: trunk/Source/WebCore/platform/blackberry/CookieManager.cpp (138567 => 138568)


--- trunk/Source/WebCore/platform/blackberry/CookieManager.cpp	2012-12-29 18:11:57 UTC (rev 138567)
+++ trunk/Source/WebCore/platform/blackberry/CookieManager.cpp	2012-12-29 19:10:56 UTC (rev 138568)
@@ -535,6 +535,11 @@
     CookieLog("CookieManager - Backingstore has %d cookies, loading them in memory now", cookies.size());
     for (size_t i = 0; i < cookies.size(); ++i) {
         ParsedCookie* newCookie = cookies[i];
+
+        // The IP flag is not persisted in the database.
+        if (BlackBerry::Platform::isIPAddress(newCookie->domain().utf8().data()))
+            newCookie->setDomain(newCookie->domain(), true);
+
         checkAndTreatCookie(newCookie, BackingStoreCookieEntry);
     }
     CookieLog("CookieManager - Backingstore loading complete.");
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to