Bug 174310 – [SOUP] Remove WebKitSoupCookieJarSqlite

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Michael Catanzaro]

It seems WebKitSoupCookieJarSqlite uses exactly the same database format as the upstream SoupCookieJarDb class. WebKitSoupCookieJarSqlite exists only because SoupCookieJarDb is relatively new, and the older SoupCookieJarSqlite class existed only in libsoup-gnome rather than libsoup proper. The advantage of continuing to use our WebKitSoupCookieJarSqlite class, besides that we already know it works, is that it utilizes the same SQLiteDatabase class that is used elsewhere in WebKit. But that's not really any reason to keep the class around, when SoupCookieJarDb does exactly the same thing. It was introduced in libsoup 2.42 and that is already our minimum required version, so we can just switch.

This passes a quick sanity-check of Epiphany's cookies dialog and also TestCookieManager.

[Michael Catanzaro]

Created attachment 314978 [details]
Patch

[Carlos Garcia Campos]

Comment on attachment 314978 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=314978&action=review

Thanks

> Source/WebKit2/ChangeLog:8
> +        It seems WebKitSoupCookieJarSqlite uses exactly the same database format as the upstream

WebKitSoupCookieJarSqlite is a copy of SoupCookieJarSqlite but using WebKit sqlite implementation and other WTF stuff.

[Michael Catanzaro]

Committed r219295: <http://trac.webkit.org/changeset/219295>