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

Unified Diff: components/certificate_transparency/single_tree_tracker.cc

Issue 2668803004: Certificate Transparency: Discard entries pending auditing on network change (Closed)
Patch Set: Created 3 years, 11 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: components/certificate_transparency/single_tree_tracker.cc
diff --git a/components/certificate_transparency/single_tree_tracker.cc b/components/certificate_transparency/single_tree_tracker.cc
index 281231690f6a3257b61ed084a75ed8d45635d57b..ea76da96864490b5e8c74c490f8a85acd3ca0b02 100644
--- a/components/certificate_transparency/single_tree_tracker.cc
+++ b/components/certificate_transparency/single_tree_tracker.cc
@@ -221,9 +221,12 @@ SingleTreeTracker::SingleTreeTracker(
weak_factory_(this) {
memory_pressure_listener_.reset(new base::MemoryPressureListener(base::Bind(
&SingleTreeTracker::OnMemoryPressure, base::Unretained(this))));
+ net::NetworkChangeNotifier::AddNetworkChangeObserver(this);
}
-SingleTreeTracker::~SingleTreeTracker() {}
+SingleTreeTracker::~SingleTreeTracker() {
+ net::NetworkChangeNotifier::RemoveNetworkChangeObserver(this);
+}
void SingleTreeTracker::OnSCTVerified(net::X509Certificate* cert,
const SignedCertificateTimestamp* sct) {
@@ -326,6 +329,11 @@ void SingleTreeTracker::NewSTHObserved(const SignedTreeHead& sth) {
ProcessPendingEntries();
}
+void SingleTreeTracker::OnNetworkChanged(
+ net::NetworkChangeNotifier::ConnectionType type) {
+ pending_entries_.clear();
+}
+
SingleTreeTracker::SCTInclusionStatus
SingleTreeTracker::GetLogEntryInclusionStatus(
net::X509Certificate* cert,

Powered by Google App Engine
This is Rietveld 408576698