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

Unified Diff: chrome/browser/safe_browsing/client_side_detection_host.cc

Issue 2540563002: Move SafeBrowsingUIManager::UnsafeResource to security_interstitials namespace (Closed)
Patch Set: rebase update 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
Index: chrome/browser/safe_browsing/client_side_detection_host.cc
diff --git a/chrome/browser/safe_browsing/client_side_detection_host.cc b/chrome/browser/safe_browsing/client_side_detection_host.cc
index ca88541a345cc9a72e0535937a9d1d5525470a51..7df4ce7ef41e04304e6e19fbb5605a113dc22cff 100644
--- a/chrome/browser/safe_browsing/client_side_detection_host.cc
+++ b/chrome/browser/safe_browsing/client_side_detection_host.cc
@@ -409,7 +409,7 @@ void ClientSideDetectionHost::DidNavigateMainFrame(
}
void ClientSideDetectionHost::OnSafeBrowsingHit(
- const SafeBrowsingUIManager::UnsafeResource& resource) {
+ const security_interstitials::UnsafeResource& resource) {
if (!web_contents())
return;
@@ -431,7 +431,7 @@ void ClientSideDetectionHost::OnSafeBrowsingHit(
// We also keep the resource around in order to be able to send the
// malicious URL to the server.
- unsafe_resource_.reset(new SafeBrowsingUIManager::UnsafeResource(resource));
+ unsafe_resource_.reset(new security_interstitials::UnsafeResource(resource));
unsafe_resource_->callback.Reset(); // Don't do anything stupid.
}
@@ -555,7 +555,7 @@ void ClientSideDetectionHost::MaybeShowPhishingWarning(GURL phishing_url,
if (is_phishing) {
DCHECK(web_contents());
if (ui_manager_.get()) {
- SafeBrowsingUIManager::UnsafeResource resource;
+ security_interstitials::UnsafeResource resource;
resource.url = phishing_url;
resource.original_url = phishing_url;
resource.is_subresource = false;
@@ -591,7 +591,7 @@ void ClientSideDetectionHost::MaybeShowMalwareWarning(GURL original_url,
if (is_malware && malware_url.is_valid() && original_url.is_valid()) {
DCHECK(web_contents());
if (ui_manager_.get()) {
- SafeBrowsingUIManager::UnsafeResource resource;
+ security_interstitials::UnsafeResource resource;
resource.url = malware_url;
resource.original_url = original_url;
resource.is_subresource = (malware_url.host() != original_url.host());

Powered by Google App Engine
This is Rietveld 408576698