Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(158)

Unified Diff: chrome/browser/browsing_data/browsing_data_remover.h

Issue 1033013003: Refactor http cache removing from a StoragePartition into separate class: (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/browsing_data/browsing_data_remover.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data/browsing_data_remover.h
diff --git a/chrome/browser/browsing_data/browsing_data_remover.h b/chrome/browser/browsing_data/browsing_data_remover.h
index 84160cbdbb796f3fd65507e555f3c8300d9507df..0e470f25fa2319684f9867681a1a9fd5fd2375ff 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.h
+++ b/chrome/browser/browsing_data/browsing_data_remover.h
@@ -23,7 +23,6 @@
#include "storage/common/quota/quota_types.h"
#include "url/gurl.h"
-class ExtensionSpecialStoragePolicy;
class IOThread;
class Profile;
@@ -36,24 +35,10 @@ class PluginDataRemover;
class StoragePartition;
}
-namespace disk_cache {
-class Backend;
-}
-
namespace net {
class URLRequestContextGetter;
}
-namespace storage {
-class QuotaManager;
-}
-
-namespace content {
-class DOMStorageContext;
-struct LocalStorageUsageInfo;
-struct SessionStorageUsageInfo;
-}
-
// BrowsingDataRemover is responsible for removing data related to browsing:
// visits in url database, downloads, cookies ...
@@ -237,15 +222,6 @@ class BrowsingDataRemover
// TODO(mkwst): See http://crbug.com/113621
friend class BrowsingDataRemoverTest;
- enum CacheState {
- STATE_NONE,
- STATE_CREATE_MAIN,
- STATE_CREATE_MEDIA,
- STATE_DELETE_MAIN,
- STATE_DELETE_MEDIA,
- STATE_DONE
- };
-
// Setter for |is_removing_|; DCHECKs that we can only start removing if we're
// not already removing, and vice-versa.
static void set_removing(bool is_removing);
@@ -332,13 +308,6 @@ class BrowsingDataRemover
// Callback for when the cache has been deleted. Invokes
// NotifyAndDeleteIfDone.
void ClearedCache();
-
- // Invoked on the IO thread to delete from the cache.
- void ClearCacheOnIOThread();
-
- // Performs the actual work to delete the cache.
- void DoClearCache(int rv);
-
#if !defined(DISABLE_NACL)
// Callback for when the NaCl cache has been deleted. Invokes
// NotifyAndDeleteIfDone.
@@ -417,9 +386,6 @@ class BrowsingDataRemover
// to artificially delay completion. Used for testing.
static CompletionInhibitor* completion_inhibitor_;
- CacheState next_cache_state_;
- disk_cache::Backend* cache_;
-
// Used to delete data from HTTP cache.
scoped_refptr<net::URLRequestContextGetter> main_context_getter_;
scoped_refptr<net::URLRequestContextGetter> media_context_getter_;
« no previous file with comments | « no previous file | chrome/browser/browsing_data/browsing_data_remover.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698