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

Unified Diff: chrome/renderer/safe_browsing/phishing_dom_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_dom_feature_extractor.cc
diff --git a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
index 17ce0e8f2fc3a4d505b5a37ee25b5528ee10a213..dd8e7c442ce7c394437892d0172ccbed6fb3a1d6 100644
--- a/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
+++ b/chrome/renderer/safe_browsing/phishing_dom_feature_extractor.cc
@@ -387,7 +387,7 @@ void PhishingDOMFeatureExtractor::ResetFrameData() {
cur_frame_data_->elements = cur_document_.all();
cur_frame_data_->domain =
net::RegistryControlledDomainService::GetDomainAndRegistry(
- cur_document_.url());
+ cur_document_.url(), net::RCDS::EXCLUDE_PRIVATE_REGISTRIES);
}
WebKit::WebDocument PhishingDOMFeatureExtractor::GetNextDocument() {
@@ -423,7 +423,7 @@ bool PhishingDOMFeatureExtractor::IsExternalDomain(const GURL& url,
domain->assign(url.host());
} else {
domain->assign(net::RegistryControlledDomainService::GetDomainAndRegistry(
- url));
+ url, net::RCDS::EXCLUDE_PRIVATE_REGISTRIES));
}
return !domain->empty() && *domain != cur_frame_data_->domain;

Powered by Google App Engine
This is Rietveld 408576698