diff options
author | mkwst@chromium.org <mkwst@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-04 10:40:55 +0000 |
---|---|---|
committer | mkwst@chromium.org <mkwst@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-06-04 10:40:55 +0000 |
commit | 3d4d4cf8cb13edd9252a40d3053850718cd6d5ae (patch) | |
tree | 98f4e64a356a856bfb15fc0cb5761ae1001dc31a /content | |
parent | 20f0609c4ed4c6eacd2d3781a13043e6d19a7293 (diff) | |
download | chromium_src-3d4d4cf8cb13edd9252a40d3053850718cd6d5ae.zip chromium_src-3d4d4cf8cb13edd9252a40d3053850718cd6d5ae.tar.gz chromium_src-3d4d4cf8cb13edd9252a40d3053850718cd6d5ae.tar.bz2 |
Teaching BrowsingDataRemover how to delete application data.
Adding a parameter to BrowsingDataRemover::Remove in order to support a
"Clear app data" flag in the "Clear Browsing Data UI". Additionally, this CL pulls the implementation of DomStorageContext::DeleteDataModifiedSince out of DomStorageContext, and into BrowsingDataRemover in order to manage the complexities of decision-making in the BDR, rather than exporting them out of chrome and into webkit.
This patch only addresses localStorage/sessionStorage and quota managed data. Cookies should already ignore the special storage policy, but we'll need to examine the remainder of the types that the BDR touches.
BUG=116372
TEST=unit_tests,test_shell_tests
Review URL: https://chromiumcodereview.appspot.com/10413072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140273 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'content')
-rw-r--r-- | content/browser/dom_storage/dom_storage_context_impl.cc | 9 | ||||
-rw-r--r-- | content/browser/dom_storage/dom_storage_context_impl.h | 1 | ||||
-rw-r--r-- | content/public/browser/dom_storage_context.h | 1 |
3 files changed, 0 insertions, 11 deletions
diff --git a/content/browser/dom_storage/dom_storage_context_impl.cc b/content/browser/dom_storage/dom_storage_context_impl.cc index 3496e42..a3ff933 100644 --- a/content/browser/dom_storage/dom_storage_context_impl.cc +++ b/content/browser/dom_storage/dom_storage_context_impl.cc @@ -170,15 +170,6 @@ void DOMStorageContextImpl::DeleteLocalStorageFile(const FilePath& file_path) { FilePathToOrigin(file_path))); } -void DOMStorageContextImpl::DeleteDataModifiedSince(const base::Time& cutoff) { - DCHECK(context_); - context_->task_runner()->PostShutdownBlockingTask( - FROM_HERE, - DomStorageTaskRunner::PRIMARY_SEQUENCE, - base::Bind(&DomStorageContext::DeleteDataModifiedSince, context_, - cutoff)); -} - void DOMStorageContextImpl::PurgeMemory() { DCHECK(context_); context_->task_runner()->PostShutdownBlockingTask( diff --git a/content/browser/dom_storage/dom_storage_context_impl.h b/content/browser/dom_storage/dom_storage_context_impl.h index 93c1af6..0ac72e7 100644 --- a/content/browser/dom_storage/dom_storage_context_impl.h +++ b/content/browser/dom_storage/dom_storage_context_impl.h @@ -41,7 +41,6 @@ class CONTENT_EXPORT DOMStorageContextImpl : virtual FilePath GetFilePath(const string16& origin_id) const OVERRIDE; virtual void DeleteForOrigin(const string16& origin_id) OVERRIDE; virtual void DeleteLocalStorageFile(const FilePath& file_path) OVERRIDE; - virtual void DeleteDataModifiedSince(const base::Time& cutoff) OVERRIDE; // Called to free up memory that's not strictly needed. void PurgeMemory(); diff --git a/content/public/browser/dom_storage_context.h b/content/public/browser/dom_storage_context.h index eb14c8c..3fb0595 100644 --- a/content/public/browser/dom_storage_context.h +++ b/content/public/browser/dom_storage_context.h @@ -40,7 +40,6 @@ class DOMStorageContext { virtual FilePath GetFilePath(const string16& origin_id) const = 0; virtual void DeleteForOrigin(const string16& origin_id) = 0; virtual void DeleteLocalStorageFile(const FilePath& file_path) = 0; - virtual void DeleteDataModifiedSince(const base::Time& cutoff) = 0; protected: virtual ~DOMStorageContext() {} |