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

Unified Diff: net/cert/sth_distributor.h

Issue 1968053002: Certificate Transparency: Notify STH Observers of known STHs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressing review comments Created 4 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 | « no previous file | net/cert/sth_distributor.cc » ('j') | net/cert/sth_distributor.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cert/sth_distributor.h
diff --git a/net/cert/sth_distributor.h b/net/cert/sth_distributor.h
index 3d9691cc56c029a47c627f6aee0987d7e2b5c3cf..246572b1d591e44b3642c04e56718bae55b89d8e 100644
--- a/net/cert/sth_distributor.h
+++ b/net/cert/sth_distributor.h
@@ -5,8 +5,11 @@
#ifndef NET_CERT_STH_DISTRIBUTOR_H_
#define NET_CERT_STH_DISTRIBUTOR_H_
+#include <vector>
+
#include "base/observer_list.h"
#include "net/base/net_export.h"
+#include "net/cert/signed_tree_head.h"
#include "net/cert/sth_observer.h"
#include "net/cert/sth_reporter.h"
@@ -28,10 +31,20 @@ class NET_EXPORT STHDistributor : public STHObserver, public STHReporter {
void NewSTHObserved(const SignedTreeHead& sth) override;
// STHReporter implementation
+ // Registers |observer| for new STH notifications and invokes the
+ // new STH notification of this |observer| with observed STHs (one
+ // per log).
void RegisterObserver(STHObserver* observer) override;
+
+ // Unregisters |observer|, should only be called with observers
+ // that were previously registered via RegisterObserver;
void UnregisterObserver(STHObserver* observer) override;
private:
+ // STHs from logs, one for each log.
+ std::vector<SignedTreeHead> observed_sths_;
Ryan Sleevi 2016/05/16 19:19:01 FWIW you can still forward declare this (because t
Eran Messeri 2016/05/17 09:51:03 Done.
+
+ // The observers for new STH notifications.
base::ObserverList<STHObserver> observer_list_;
};
« no previous file with comments | « no previous file | net/cert/sth_distributor.cc » ('j') | net/cert/sth_distributor.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698