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

Unified Diff: chrome/renderer/safe_browsing/phishing_url_feature_extractor.cc

Issue 13979002: Add support for split PSL list distinctions. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed chrome_frame compilation issue Created 7 years, 8 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/renderer/safe_browsing/phishing_url_feature_extractor.cc
diff --git a/chrome/renderer/safe_browsing/phishing_url_feature_extractor.cc b/chrome/renderer/safe_browsing/phishing_url_feature_extractor.cc
index 4e438a99efe5b57bc9b04a82b6b3174fc516aaa2..8c85d3918f645ba79fa656a609bb11465ceccbc1 100644
--- a/chrome/renderer/safe_browsing/phishing_url_feature_extractor.cc
+++ b/chrome/renderer/safe_browsing/phishing_url_feature_extractor.cc
@@ -39,7 +39,10 @@ bool PhishingUrlFeatureExtractor::ExtractFeatures(const GURL& url,
// Disallow unknown registries so that we don't classify
// partial hostnames (e.g. "www.subdomain").
size_t registry_length =
- net::RegistryControlledDomainService::GetRegistryLength(host, false);
+ net::RegistryControlledDomainService::GetRegistryLength(
+ host,
+ net::RCDS::EXCLUDE_UNKNOWN_REGISTRIES,
+ net::RCDS::EXCLUDE_PRIVATE_REGISTRIES);
if (registry_length == 0 || registry_length == std::string::npos) {
DVLOG(1) << "Could not find TLD for host: " << host;

Powered by Google App Engine
This is Rietveld 408576698