Bug 213337 – [GTK][WPE] Add API to configure and handle DOM cache to WebKitWebsiteDataManager

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

[Carlos Garcia Campos]

The default path is always used.

[Carlos Garcia Campos]

Created attachment 402201 [details]
Patch

[EWS Watchlist]

Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API

[Adrian Perez]

Comment on attachment 402201 [details]
Patch

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

> Source/WebKit/UIProcess/API/glib/WebKitWebsiteDataManager.cpp:241
> +            priv->domCacheDirectory.reset(g_build_filename(priv->baseCacheDirectory.get(), "CacheStorage", nullptr));

I wonder why this is “CamelCase” while the other directories
are “allsmallcaps” 🤔️

[Carlos Garcia Campos]

(In reply to Adrian Perez from comment #3)
> Comment on attachment 402201 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=402201&action=review
> 
> > Source/WebKit/UIProcess/API/glib/WebKitWebsiteDataManager.cpp:241
> > +            priv->domCacheDirectory.reset(g_build_filename(priv->baseCacheDirectory.get(), "CacheStorage", nullptr));
> 
> I wonder why this is “CamelCase” while the other directories
> are “allsmallcaps” 🤔️

Because it's consistent with the default directory name.

[Adrian Perez]

(In reply to Carlos Garcia Campos from comment #4)
> (In reply to Adrian Perez from comment #3)
> > Comment on attachment 402201 [details]
> > Patch
> > 
> > View in context:
> > https://bugs.webkit.org/attachment.cgi?id=402201&action=review
> > 
> > > Source/WebKit/UIProcess/API/glib/WebKitWebsiteDataManager.cpp:241
> > > +            priv->domCacheDirectory.reset(g_build_filename(priv->baseCacheDirectory.get(), "CacheStorage", nullptr));
> > 
> > I wonder why this is “CamelCase” while the other directories
> > are “allsmallcaps” 🤔️
> 
> Because it's consistent with the default directory name.

Aye, no more questions then, let's land this 👍️

[Carlos Garcia Campos]

Committed r263386: <https://trac.webkit.org/changeset/263386>