Index: content/browser/storage_partition_impl.cc |
diff --git a/content/browser/storage_partition_impl.cc b/content/browser/storage_partition_impl.cc |
index 52497c902953e3280b480456e82ee548641c3cbf..1c2b2c2e88ef008d5e69a12ab3b607a192d9bf6f 100644 |
--- a/content/browser/storage_partition_impl.cc |
+++ b/content/browser/storage_partition_impl.cc |
@@ -51,25 +51,24 @@ void OnClearedCookies(const base::Closure& callback, int num_deleted) { |
callback.Run(); |
} |
+// Cookie matcher and storage_origin are never both populated. |
void ClearCookiesOnIOThread( |
const scoped_refptr<net::URLRequestContextGetter>& rq_context, |
- const base::Time begin, |
- const base::Time end, |
- const GURL& storage_origin, |
+ const base::Time begin, const base::Time end, const GURL& storage_origin, |
+ const StoragePartition::CookieMatcherFunction& cookie_matcher, |
const base::Closure& callback) { |
DCHECK_CURRENTLY_ON(BrowserThread::IO); |
- net::CookieStore* cookie_store = rq_context-> |
- GetURLRequestContext()->cookie_store(); |
- if (storage_origin.is_empty()) { |
- cookie_store->DeleteAllCreatedBetweenAsync( |
- begin, |
- end, |
- base::Bind(&OnClearedCookies, callback)); |
- } else { |
+ net::CookieStore* cookie_store = |
+ rq_context->GetURLRequestContext()->cookie_store(); |
+ if (!cookie_matcher.is_null()) { |
+ cookie_store->DeleteAllCreatedBetweenWithPredicateAsync( |
+ begin, end, cookie_matcher, base::Bind(&OnClearedCookies, callback)); |
+ } else if (!storage_origin.is_empty()) { |
cookie_store->DeleteAllCreatedBetweenForHostAsync( |
- begin, |
- end, |
- storage_origin, base::Bind(&OnClearedCookies, callback)); |
+ begin, end, storage_origin, base::Bind(&OnClearedCookies, callback)); |
+ } else { |
+ cookie_store->DeleteAllCreatedBetweenAsync( |
+ begin, end, base::Bind(&OnClearedCookies, callback)); |
} |
} |
@@ -247,6 +246,7 @@ struct StoragePartitionImpl::QuotaManagedDataDeletionHelper { |
const StoragePartition::OriginMatcherFunction& origin_matcher); |
void ClearOriginsOnIOThread( |
+ |
storage::QuotaManager* quota_manager, |
const scoped_refptr<storage::SpecialStoragePolicy>& |
special_storage_policy, |
@@ -288,6 +288,7 @@ struct StoragePartitionImpl::DataDeletionHelper { |
void ClearDataOnUIThread( |
const GURL& storage_origin, |
const OriginMatcherFunction& origin_matcher, |
+ const CookieMatcherFunction& cookie_matcher, |
const base::FilePath& path, |
net::URLRequestContextGetter* rq_context, |
DOMStorageContextWrapper* dom_storage_context, |
@@ -601,6 +602,7 @@ void StoragePartitionImpl::ClearDataImpl( |
uint32_t quota_storage_remove_mask, |
const GURL& storage_origin, |
const OriginMatcherFunction& origin_matcher, |
+ const CookieMatcherFunction& cookie_matcher, |
net::URLRequestContextGetter* rq_context, |
const base::Time begin, |
const base::Time end, |
@@ -611,16 +613,10 @@ void StoragePartitionImpl::ClearDataImpl( |
callback); |
// |helper| deletes itself when done in |
// DataDeletionHelper::DecrementTaskCountOnUI(). |
- helper->ClearDataOnUIThread(storage_origin, |
- origin_matcher, |
- GetPath(), |
- rq_context, |
- dom_storage_context_.get(), |
- quota_manager_.get(), |
- special_storage_policy_.get(), |
- webrtc_identity_store_.get(), |
- begin, |
- end); |
+ helper->ClearDataOnUIThread( |
+ storage_origin, origin_matcher, cookie_matcher, GetPath(), rq_context, |
+ dom_storage_context_.get(), quota_manager_.get(), |
+ special_storage_policy_.get(), webrtc_identity_store_.get(), begin, end); |
} |
void StoragePartitionImpl:: |
@@ -765,6 +761,7 @@ void StoragePartitionImpl::DataDeletionHelper::DecrementTaskCountOnUI() { |
void StoragePartitionImpl::DataDeletionHelper::ClearDataOnUIThread( |
const GURL& storage_origin, |
const OriginMatcherFunction& origin_matcher, |
+ const CookieMatcherFunction& cookie_matcher, |
const base::FilePath& path, |
net::URLRequestContextGetter* rq_context, |
DOMStorageContextWrapper* dom_storage_context, |
@@ -785,8 +782,8 @@ void StoragePartitionImpl::DataDeletionHelper::ClearDataOnUIThread( |
IncrementTaskCountOnUI(); |
BrowserThread::PostTask( |
BrowserThread::IO, FROM_HERE, |
- base::Bind(&ClearCookiesOnIOThread, |
- make_scoped_refptr(rq_context), begin, end, storage_origin, |
+ base::Bind(&ClearCookiesOnIOThread, make_scoped_refptr(rq_context), |
+ begin, end, storage_origin, cookie_matcher, |
decrement_callback)); |
} |
@@ -861,13 +858,9 @@ void StoragePartitionImpl::ClearDataForOrigin( |
net::URLRequestContextGetter* request_context_getter, |
const base::Closure& callback) { |
DCHECK_CURRENTLY_ON(BrowserThread::UI); |
- ClearDataImpl(remove_mask, |
- quota_storage_remove_mask, |
- storage_origin, |
- OriginMatcherFunction(), |
- request_context_getter, |
- base::Time(), |
- base::Time::Max(), |
+ ClearDataImpl(remove_mask, quota_storage_remove_mask, storage_origin, |
+ OriginMatcherFunction(), CookieMatcherFunction(), |
+ request_context_getter, base::Time(), base::Time::Max(), |
callback); |
} |
@@ -880,7 +873,20 @@ void StoragePartitionImpl::ClearData( |
const base::Time end, |
const base::Closure& callback) { |
ClearDataImpl(remove_mask, quota_storage_remove_mask, storage_origin, |
- origin_matcher, GetURLRequestContext(), begin, end, callback); |
+ origin_matcher, CookieMatcherFunction(), GetURLRequestContext(), |
+ begin, end, callback); |
+} |
+ |
+void StoragePartitionImpl::ClearData( |
+ uint32_t remove_mask, |
+ uint32_t quota_storage_remove_mask, |
+ const OriginMatcherFunction& origin_matcher, |
+ const CookieMatcherFunction& cookie_matcher, |
+ const base::Time begin, |
+ const base::Time end, |
+ const base::Closure& callback) { |
+ ClearDataImpl(remove_mask, quota_storage_remove_mask, GURL(), origin_matcher, |
+ cookie_matcher, GetURLRequestContext(), begin, end, callback); |
} |
void StoragePartitionImpl::Flush() { |