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

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

Issue 2647683002: Consolidate Origin- and RegistrableDomain- FilterBuilder into one class (Closed)
Patch Set: Rebase. Created 3 years, 11 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/browsing_data_remover_impl.h
diff --git a/chrome/browser/browsing_data/browsing_data_remover_impl.h b/chrome/browser/browsing_data/browsing_data_remover_impl.h
index 9397d0468b83663676b96b00fa8c7aceac53e929..ff6c3327d50e99c901dcc52d45b57d8dafdeae47 100644
--- a/chrome/browser/browsing_data/browsing_data_remover_impl.h
+++ b/chrome/browser/browsing_data/browsing_data_remover_impl.h
@@ -149,15 +149,6 @@ class BrowsingDataRemoverImpl :
// Testing the private RemovalTask.
FRIEND_TEST_ALL_PREFIXES(BrowsingDataRemoverImplTest, MultipleTasks);
- // The BrowsingDataRemover tests need to be able to access the implementation
- // of Remove(), as it exposes details that aren't yet available in the public
- // API. As soon as those details are exposed via new methods, this should be
- // removed.
- //
- // TODO(mkwst): See http://crbug.com/113621
- friend class BrowsingDataRemoverImplTest;
- friend class ChromeBrowsingDataRemoverDelegateTest;
-
friend class BrowsingDataRemoverFactory;
// Represents a single removal task. Contains all parameters needed to execute

Powered by Google App Engine
This is Rietveld 408576698