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

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

Issue 2261673002: Move static RegistrableDomainFilterBuilder methods to an anonymous namespace (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: BindRepeating Created 4 years 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
« no previous file with comments | « no previous file | chrome/browser/browsing_data/registrable_domain_filter_builder.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browsing_data/registrable_domain_filter_builder.h
diff --git a/chrome/browser/browsing_data/registrable_domain_filter_builder.h b/chrome/browser/browsing_data/registrable_domain_filter_builder.h
index 00bcafa16fc7de3833a255503b5f86e21f4a7579..1f7d1869bf27ff2643e342dbe94209abb7524f38 100644
--- a/chrome/browser/browsing_data/registrable_domain_filter_builder.h
+++ b/chrome/browser/browsing_data/registrable_domain_filter_builder.h
@@ -97,48 +97,9 @@ class RegistrableDomainFilterBuilder : public BrowsingDataFilterBuilder {
bool IsEmpty() const override;
private:
- // True if the domain of |url| is in the whitelist, or isn't in the blacklist.
- // The whitelist or blacklist is represented as |registerable_domains|
- // and |mode|.
- static bool MatchesURL(std::set<std::string>* registerable_domains,
- Mode mode,
- const GURL& url);
-
- // True if the pattern something in the whitelist, or doesn't match something
- // in the blacklist.
- // The whitelist or blacklist is represented as |patterns|, and |mode|.
- static bool MatchesWebsiteSettingsPattern(
- std::vector<ContentSettingsPattern>* patterns,
- Mode mode,
- const ContentSettingsPattern& pattern);
-
- // True if no domains can see the given cookie and we're a blacklist, or any
- // domains can see the cookie and we're a whitelist.
- // The whitelist or blacklist is represented as |domains_and_ips| and |mode|.
- static bool MatchesCookieForRegisterableDomainsAndIPs(
- std::set<std::string>* domains_and_ips,
- Mode mode,
- const net::CanonicalCookie& cookie);
-
- // True if none of the supplied domains matches this Channel ID's server ID
- // and we're a blacklist, or one of them does and we're a whitelist.
- // The whitelist or blacklist is represented as |domains_and_ips| and |mode|.
- static bool MatchesChannelIDForRegisterableDomainsAndIPs(
- std::set<std::string>* domains_and_ips,
- Mode mode,
- const std::string& channel_id_server_id);
-
- // True if none of the supplied domains matches this plugin's |site| and we're
- // a blacklist, or one of them does and we're a whitelist. The whitelist or
- // blacklist is represented by |domains_and_ips| and |mode|.
- static bool MatchesPluginSiteForRegisterableDomainsAndIPs(
- std::set<std::string>* domains_and_ips,
- Mode mode,
- const std::string& site);
-
- // The list of domains and whether they should be interpreted as a whitelist
- // or blacklist.
- std::set<std::string> domain_list_;
+ // The set of domains that constitute the whitelist or the blacklist,
+ // depending on mode().
+ std::set<std::string> domains_;
DISALLOW_COPY_AND_ASSIGN(RegistrableDomainFilterBuilder);
};
« no previous file with comments | « no previous file | chrome/browser/browsing_data/registrable_domain_filter_builder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698