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

Unified Diff: chrome/browser/profiles/profile_io_data.cc

Issue 1579063002: Implement a skeleton version of Expect CT reports (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: remove unnecessary (?) NET_EXPORTs Created 4 years, 9 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/browser/profiles/profile_io_data.cc
diff --git a/chrome/browser/profiles/profile_io_data.cc b/chrome/browser/profiles/profile_io_data.cc
index 9e86ca9adb1a8e8c9ac1ea37b4657a8e031ea9d5..b687edfac02e865bda376fcfcdcc3e85a06db5cd 100644
--- a/chrome/browser/profiles/profile_io_data.cc
+++ b/chrome/browser/profiles/profile_io_data.cc
@@ -45,6 +45,7 @@
#include "chrome/browser/predictors/resource_prefetch_predictor_factory.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ssl/chrome_expect_ct_reporter.h"
#include "chrome/browser/ui/search/new_tab_page_interceptor_service.h"
#include "chrome/browser/ui/search/new_tab_page_interceptor_service_factory.h"
#include "chrome/common/chrome_paths.h"
@@ -666,6 +667,10 @@ ProfileIOData::~ProfileIOData() {
transport_security_state_->SetReportSender(nullptr);
certificate_report_sender_.reset();
+ if (transport_security_state_)
+ transport_security_state_->SetExpectCTReporter(nullptr);
+ expect_ct_reporter_.reset();
+
// TODO(ajwong): These AssertNoURLRequests() calls are unnecessary since they
// are already done in the URLRequestContext destructor.
if (main_request_context_)
@@ -1072,6 +1077,10 @@ void ProfileIOData::Init(
net::CertificateReportSender::DO_NOT_SEND_COOKIES));
transport_security_state_->SetReportSender(certificate_report_sender_.get());
+ expect_ct_reporter_.reset(
+ new ChromeExpectCTReporter(main_request_context_.get()));
+ transport_security_state_->SetExpectCTReporter(expect_ct_reporter_.get());
+
// Take ownership over these parameters.
cookie_settings_ = profile_params_->cookie_settings;
host_content_settings_map_ = profile_params_->host_content_settings_map;

Powered by Google App Engine
This is Rietveld 408576698