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

Unified Diff: chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc

Issue 2898443002: Implement GetBrowsingDataRemoverDelegate() for BrowserContext subclasses (Closed)
Patch Set: More fixes. Created 3 years, 7 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: chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc
diff --git a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc
index b642b8190fd5b2cd0d4a7c85a33ca1372287e7e0..6f375c87e6dda2280d9a3b1bee684ddf0bb93107 100644
--- a/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc
+++ b/chrome/browser/browsing_data/chrome_browsing_data_remover_delegate_unittest.cc
@@ -751,21 +751,6 @@ class RemoveDownloadsTester {
DISALLOW_COPY_AND_ASSIGN(RemoveDownloadsTester);
};
-// TestingProfile does not contain ChromeBrowsingDataRemoverDelegate. Add it
-// for the purpose of this test.
-class TestingProfileWithDelegate : public TestingProfile {
- public:
- TestingProfileWithDelegate() : delegate_(this) {}
-
- content::BrowsingDataRemoverDelegate* GetBrowsingDataRemoverDelegate()
- override {
- return &delegate_;
- }
-
- private:
- ChromeBrowsingDataRemoverDelegate delegate_;
-};
-
} // namespace
// RemoveAutofillTester is not a part of the anonymous namespace above, as
@@ -860,7 +845,7 @@ class RemoveAutofillTester : public autofill::PersonalDataManagerObserver {
class ChromeBrowsingDataRemoverDelegateTest : public testing::Test {
public:
ChromeBrowsingDataRemoverDelegateTest()
- : profile_(new TestingProfileWithDelegate()),
+ : profile_(new TestingProfile()),
clear_domain_reliability_tester_(profile_.get()) {
remover_ = content::BrowserContext::GetBrowsingDataRemover(profile_.get());
« no previous file with comments | « android_webview/browser/aw_browser_context.cc ('k') | chrome/browser/profiles/off_the_record_profile_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698