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

Unified Diff: content/public/browser/browser_context.h

Issue 2827523003: Move BrowsingDataRemover to content/ (Closed)
Patch Set: Addressed some comments. Created 3 years, 8 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
Index: content/public/browser/browser_context.h
diff --git a/content/public/browser/browser_context.h b/content/public/browser/browser_context.h
index 89461daa21f4788658306876b3e21fd79e7a810f..a5a3db690cc7edad8b11510c7610cc460282c622 100644
--- a/content/public/browser/browser_context.h
+++ b/content/public/browser/browser_context.h
@@ -52,6 +52,8 @@ namespace content {
class BackgroundSyncController;
class BlobHandle;
class BrowserPluginGuestManager;
+class BrowsingDataRemover;
+class BrowsingDataRemoverDelegate;
class DownloadManager;
class DownloadManagerDelegate;
class PermissionManager;
@@ -84,6 +86,11 @@ class CONTENT_EXPORT BrowserContext : public base::SupportsUserData {
// mount points. Currenty, non-nullptr value is returned only on ChromeOS.
static storage::ExternalMountPoints* GetMountPoints(BrowserContext* context);
+ // Returns a BrowsingDataRemover that can schedule data deletion tasks
+ // for this |context|.
+ static content::BrowsingDataRemover* GetBrowsingDataRemover(
jam 2017/04/21 22:58:31 nit: no content::
msramek 2017/04/25 01:29:10 Done. Here and in the old code below.
+ BrowserContext* context);
+
static content::StoragePartition* GetStoragePartition(
BrowserContext* browser_context, SiteInstance* site_instance);
static content::StoragePartition* GetStoragePartitionForSite(
@@ -222,6 +229,10 @@ class CONTENT_EXPORT BrowserContext : public base::SupportsUserData {
// nullptr otherwise.
virtual BackgroundSyncController* GetBackgroundSyncController() = 0;
+ // Returns the BrowsingDataRemoverDelegate for this context. This will be
+ // called once per context. It's valid to return nullptr.
+ virtual BrowsingDataRemoverDelegate* GetBrowsingDataRemoverDelegate();
+
// Creates the main net::URLRequestContextGetter. It's called only once.
virtual net::URLRequestContextGetter* CreateRequestContext(
ProtocolHandlerMap* protocol_handlers,

Powered by Google App Engine
This is Rietveld 408576698