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

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

Issue 6935032: Include full redirect chain in downloads safebrowsing ping. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: initialize url_chain in tests Created 9 years, 7 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
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_service.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/safe_browsing/safe_browsing_service.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_service.cc b/chrome/browser/safe_browsing/safe_browsing_service.cc
index 0d9caf1eaef56572a58c8ba43644dd147f2a6d59..9aceb11e4501b5a39c15b8704852caa31f2f7212 100644
--- a/chrome/browser/safe_browsing/safe_browsing_service.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_service.cc
@@ -1026,7 +1026,7 @@ void SafeBrowsingService::DoDisplayBlockingPage(
page_url = resource.original_url;
}
ReportSafeBrowsingHit(resource.url, page_url, referrer_url, is_subresource,
- resource.threat_type);
+ resource.threat_type, std::string() /* post_data */);
}
SafeBrowsingBlockingPage::ShowBlockingPage(this, resource);
@@ -1039,7 +1039,8 @@ void SafeBrowsingService::ReportSafeBrowsingHit(
const GURL& page_url,
const GURL& referrer_url,
bool is_subresource,
- SafeBrowsingService::UrlCheckResult threat_type) {
+ SafeBrowsingService::UrlCheckResult threat_type,
+ const std::string& post_data) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
if (!CanReportStats())
return;
@@ -1053,7 +1054,8 @@ void SafeBrowsingService::ReportSafeBrowsingHit(
page_url,
referrer_url,
is_subresource,
- threat_type));
+ threat_type,
+ post_data));
}
void SafeBrowsingService::ReportSafeBrowsingHitOnIOThread(
@@ -1061,7 +1063,8 @@ void SafeBrowsingService::ReportSafeBrowsingHitOnIOThread(
const GURL& page_url,
const GURL& referrer_url,
bool is_subresource,
- SafeBrowsingService::UrlCheckResult threat_type) {
+ SafeBrowsingService::UrlCheckResult threat_type,
+ const std::string& post_data) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
if (!enabled_)
return;
@@ -1071,7 +1074,7 @@ void SafeBrowsingService::ReportSafeBrowsingHitOnIOThread(
<< threat_type;
protocol_manager_->ReportSafeBrowsingHit(malicious_url, page_url,
referrer_url, is_subresource,
- threat_type);
+ threat_type, post_data);
}
// If the user had opted-in to send MalwareDetails, this gets called
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_service.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698