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

Side by Side Diff: chrome/browser/browsing_data_remover.h

Issue 9419033: Move creation of BrowserContext objects that live in content to content, instead of depending on th… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Fix memory leaks in tests Created 8 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_BROWSING_DATA_REMOVER_H_ 5 #ifndef CHROME_BROWSER_BROWSING_DATA_REMOVER_H_
6 #define CHROME_BROWSER_BROWSING_DATA_REMOVER_H_ 6 #define CHROME_BROWSER_BROWSING_DATA_REMOVER_H_
7 #pragma once 7 #pragma once
8 8
9 #include <set> 9 #include <set>
10 10
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 void AddObserver(Observer* observer); 123 void AddObserver(Observer* observer);
124 void RemoveObserver(Observer* observer); 124 void RemoveObserver(Observer* observer);
125 125
126 // Called when history deletion is done. 126 // Called when history deletion is done.
127 void OnHistoryDeletionDone(); 127 void OnHistoryDeletionDone();
128 128
129 // Quota managed data uses a different bitmask for types than 129 // Quota managed data uses a different bitmask for types than
130 // BrowsingDataRemover uses. This method generates that mask. 130 // BrowsingDataRemover uses. This method generates that mask.
131 static int GenerateQuotaClientMask(int remove_mask); 131 static int GenerateQuotaClientMask(int remove_mask);
132 132
133 // Used for testing.
134 void OverrideQuotaManagerForTesting(quota::QuotaManager* quota_manager);
135
133 static bool is_removing() { return removing_; } 136 static bool is_removing() { return removing_; }
134 137
135 private: 138 private:
136 // The clear API needs to be able to toggle removing_ in order to test that 139 // The clear API needs to be able to toggle removing_ in order to test that
137 // only one BrowsingDataRemover instance can be called at a time. 140 // only one BrowsingDataRemover instance can be called at a time.
138 FRIEND_TEST_ALL_PREFIXES(ExtensionBrowsingDataTest, OneAtATime); 141 FRIEND_TEST_ALL_PREFIXES(ExtensionBrowsingDataTest, OneAtATime);
139 142
140 // The BrowsingDataRemover tests need to be able to access the implementation 143 // The BrowsingDataRemover tests need to be able to access the implementation
141 // of Remove(), as it exposes details that aren't yet available in the public 144 // of Remove(), as it exposes details that aren't yet available in the public
142 // API. As soon as those details are exposed via new methods, this should be 145 // API. As soon as those details are exposed via new methods, this should be
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 317
315 ObserverList<Observer> observer_list_; 318 ObserverList<Observer> observer_list_;
316 319
317 // Used if we need to clear history. 320 // Used if we need to clear history.
318 CancelableRequestConsumer request_consumer_; 321 CancelableRequestConsumer request_consumer_;
319 322
320 DISALLOW_COPY_AND_ASSIGN(BrowsingDataRemover); 323 DISALLOW_COPY_AND_ASSIGN(BrowsingDataRemover);
321 }; 324 };
322 325
323 #endif // CHROME_BROWSER_BROWSING_DATA_REMOVER_H_ 326 #endif // CHROME_BROWSER_BROWSING_DATA_REMOVER_H_
OLDNEW
« no previous file with comments | « chrome/browser/browsing_data_quota_helper_impl.cc ('k') | chrome/browser/browsing_data_remover.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698