<html>
<head>
<base href="https://bugs.webkit.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add API to WebKitWebsiteDataManager to handle website data"
href="https://bugs.webkit.org/show_bug.cgi?id=146589#c24">Comment # 24</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [GTK] Add API to WebKitWebsiteDataManager to handle website data"
href="https://bugs.webkit.org/show_bug.cgi?id=146589">bug 146589</a>
from <span class="vcard"><a class="email" href="mailto:cgarcia@igalia.com" title="Carlos Garcia Campos <cgarcia@igalia.com>"> <span class="fn">Carlos Garcia Campos</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=146589#c23">comment #23</a>)
<span class="quote">> Comment on <span class=""><a href="attachment.cgi?id=299429&action=diff" name="attach_299429" title="New patch">attachment 299429</a> <a href="attachment.cgi?id=299429&action=edit" title="New patch">[details]</a></span>
> New patch
>
> View in context:
> <a href="https://bugs.webkit.org/attachment.cgi?id=299429&action=review">https://bugs.webkit.org/attachment.cgi?id=299429&action=review</a>
>
> > Source/WebKit2/ChangeLog:33
> > + (webkit_website_data_manager_remove): Remove the website data of the given types for the given WebKitWebsiteData list.
> > + (webkit_website_data_manager_remove_finish):
> > + (webkit_website_data_manager_clear): Clear all website data of the given types modified since the given time span.
> > + (webkit_website_data_manager_clear_finish):
>
> I realized the distinction between _remove and _clear is too confusing. How
> about renaming _clear to _clear_all?</span >
That's why it's documented :-) I find clear_all redundant and even more confusing, all what? all data types? of course that's why you pass the types to clear, all websites data? of course since you are not providing any. Doesn't clear implies all in English?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>