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

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

Issue 2503243003: Wire up CertificateReportingService to handle report uploads (Closed)
Patch Set: Rebase 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/ping_manager.cc
diff --git a/chrome/browser/safe_browsing/ping_manager.cc b/chrome/browser/safe_browsing/ping_manager.cc
index 3027d2096537db364574c13ef1a59d164dd0f280..137d884938fca01e4343849484d6bd5e47ec5940 100644
--- a/chrome/browser/safe_browsing/ping_manager.cc
+++ b/chrome/browser/safe_browsing/ping_manager.cc
@@ -99,10 +99,9 @@ SafeBrowsingPingManager::SafeBrowsingPingManager(
DCHECK(!url_prefix_.empty());
if (request_context_getter) {
- net::ReportSender::CookiesPreference cookies_preference;
- GURL certificate_upload_url;
- cookies_preference = net::ReportSender::DO_NOT_SEND_COOKIES;
- certificate_upload_url = GURL(kExtendedReportingUploadUrlInsecure);
+ const net::ReportSender::CookiesPreference cookies_preference =
+ net::ReportSender::DO_NOT_SEND_COOKIES;
+ const GURL certificate_upload_url(kExtendedReportingUploadUrlInsecure);
certificate_error_reporter_.reset(new certificate_reporting::ErrorReporter(
request_context_getter->GetURLRequestContext(), certificate_upload_url,

Powered by Google App Engine
This is Rietveld 408576698