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

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

Issue 2292443003: Support host-based deletion for SSLHostStateDelegate (Closed)
Patch Set: Created 4 years, 4 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.h
diff --git a/chrome/browser/browsing_data/browsing_data_remover.h b/chrome/browser/browsing_data/browsing_data_remover.h
index 3c2ecb0b5b0d84cd1573bcdbbe4ed25e3b941ec4..d0ac1fec621530d7e334064ff990c369e260b151 100644
--- a/chrome/browser/browsing_data/browsing_data_remover.h
+++ b/chrome/browser/browsing_data/browsing_data_remover.h
@@ -331,8 +331,6 @@ class BrowsingDataRemover : public KeyedService
// The clear API needs to be able to toggle removing_ in order to test that
// only one BrowsingDataRemover instance can be called at a time.
FRIEND_TEST_ALL_PREFIXES(ExtensionBrowsingDataTest, OneAtATime);
- // Testing our static method, ClearSettingsForOneTypeWithPredicate.
- FRIEND_TEST_ALL_PREFIXES(BrowsingDataRemoverTest, ClearWithPredicate);
// Testing the private RemovalTask.
FRIEND_TEST_ALL_PREFIXES(BrowsingDataRemoverTest, MultipleTasks);
@@ -363,17 +361,6 @@ class BrowsingDataRemover : public KeyedService
Observer* observer;
};
- // Clears all host-specific settings for one content type that satisfy the
- // given predicate.
- //
- // This should only be called on the UI thread.
- static void ClearSettingsForOneTypeWithPredicate(
- HostContentSettingsMap* content_settings_map,
- ContentSettingsType content_type,
- const base::Callback<
- bool(const ContentSettingsPattern& primary_pattern,
- const ContentSettingsPattern& secondary_pattern)>& predicate);
-
void Shutdown() override;
// Setter for |is_removing_|; DCHECKs that we can only start removing if we're

Powered by Google App Engine
This is Rietveld 408576698