Title: [134615] trunk/Source/WebCore
Revision
134615
Author
[email protected]
Date
2012-11-14 08:53:06 -0800 (Wed, 14 Nov 2012)

Log Message

[BlackBerry] Updating BB Cookie database to use WAL
https://bugs.webkit.org/show_bug.cgi?id=102237

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

The cookie database is accessed by one process only and should be updated to
use the WAL journal mode for better I/O performance.

PR 236553

cookieCollection.db-wal is created after the conversion to WAL.
Tested cookie persistence by logging on to random sites and restarting the browser and
check if it automatically logs in.
Also tested using Opera's cookie persistence test.

* platform/blackberry/CookieDatabaseBackingStore/CookieDatabaseBackingStore.cpp:
(WebCore::CookieDatabaseBackingStore::invokeOpen):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (134614 => 134615)


--- trunk/Source/WebCore/ChangeLog	2012-11-14 16:31:03 UTC (rev 134614)
+++ trunk/Source/WebCore/ChangeLog	2012-11-14 16:53:06 UTC (rev 134615)
@@ -1,3 +1,23 @@
+2012-11-14  Otto Derek Cheung  <[email protected]>
+
+        [BlackBerry] Updating BB Cookie database to use WAL
+        https://bugs.webkit.org/show_bug.cgi?id=102237
+
+        Reviewed by Rob Buis.
+
+        The cookie database is accessed by one process only and should be updated to
+        use the WAL journal mode for better I/O performance.
+
+        PR 236553
+
+        cookieCollection.db-wal is created after the conversion to WAL.
+        Tested cookie persistence by logging on to random sites and restarting the browser and
+        check if it automatically logs in.
+        Also tested using Opera's cookie persistence test.
+
+        * platform/blackberry/CookieDatabaseBackingStore/CookieDatabaseBackingStore.cpp:
+        (WebCore::CookieDatabaseBackingStore::invokeOpen):
+
 2012-11-14  Erik Arvidsson  <[email protected]>
 
         Update DOMException name: IndexSizeError

Modified: trunk/Source/WebCore/platform/blackberry/CookieDatabaseBackingStore/CookieDatabaseBackingStore.cpp (134614 => 134615)


--- trunk/Source/WebCore/platform/blackberry/CookieDatabaseBackingStore/CookieDatabaseBackingStore.cpp	2012-11-14 16:31:03 UTC (rev 134614)
+++ trunk/Source/WebCore/platform/blackberry/CookieDatabaseBackingStore/CookieDatabaseBackingStore.cpp	2012-11-14 16:53:06 UTC (rev 134615)
@@ -112,7 +112,7 @@
     }
 
     m_db.executeCommand("PRAGMA locking_mode=EXCLUSIVE;");
-    m_db.executeCommand("PRAGMA journal_mode=TRUNCATE;");
+    m_db.executeCommand("PRAGMA journal_mode=WAL;");
 
     const String primaryKeyFields("PRIMARY KEY (protocol, host, path, name)");
     const String databaseFields("name TEXT, value TEXT, host TEXT, path TEXT, expiry DOUBLE, lastAccessed DOUBLE, isSecure INTEGER, isHttpOnly INTEGER, creationTime DOUBLE, protocol TEXT");
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to