Title: [230619] trunk/Source/WebCore
Revision
230619
Author
commit-qu...@webkit.org
Date
2018-04-12 18:27:54 -0700 (Thu, 12 Apr 2018)

Log Message

[Curl] Cookie Database on memory path should be :memory:
https://bugs.webkit.org/show_bug.cgi?id=184572

Patch by Christopher Reid <chris.r...@sony.com> on 2018-04-12
Reviewed by Alex Christensen.

CookieJarDB::isOnMemory is comparing the database path with :onmemory: but it should be using :memory:
For reference, see: https://www.sqlite.org/inmemorydb.html

* platform/network/curl/CookieJarDB.h:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (230618 => 230619)


--- trunk/Source/WebCore/ChangeLog	2018-04-13 01:09:50 UTC (rev 230618)
+++ trunk/Source/WebCore/ChangeLog	2018-04-13 01:27:54 UTC (rev 230619)
@@ -1,3 +1,15 @@
+2018-04-12  Christopher Reid  <chris.r...@sony.com>
+
+        [Curl] Cookie Database on memory path should be :memory:
+        https://bugs.webkit.org/show_bug.cgi?id=184572
+
+        Reviewed by Alex Christensen.
+
+        CookieJarDB::isOnMemory is comparing the database path with :onmemory: but it should be using :memory:
+        For reference, see: https://www.sqlite.org/inmemorydb.html
+
+        * platform/network/curl/CookieJarDB.h:
+
 2018-04-12  Daniel Bates  <daba...@apple.com>
 
         Attmept to fix the Windows build following <https://trac.webkit.org/changeset/230602/>

Modified: trunk/Source/WebCore/platform/network/curl/CookieJarDB.h (230618 => 230619)


--- trunk/Source/WebCore/platform/network/curl/CookieJarDB.h	2018-04-13 01:09:50 UTC (rev 230618)
+++ trunk/Source/WebCore/platform/network/curl/CookieJarDB.h	2018-04-13 01:27:54 UTC (rev 230619)
@@ -62,7 +62,7 @@
 
     bool m_detectedDatabaseCorruption {false};
 
-    bool isOnMemory() const { return (m_databasePath == ":onmemory:"); };
+    bool isOnMemory() const { return (m_databasePath == ":memory:"); };
 
     bool openDatabase();
     void closeDatabase();
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to