Certificate Transparency: Notify STH Observers of known STHs

net/cert/sth_distributor.h

Index: net/cert/sth_distributor.h
diff --git a/net/cert/sth_distributor.h b/net/cert/sth_distributor.h
index 3d9691cc56c029a47c627f6aee0987d7e2b5c3cf..28864ceb01a4b1e7f70a33d9c9284b9d573f0d51 100644
--- a/net/cert/sth_distributor.h
+++ b/net/cert/sth_distributor.h
@@ -5,6 +5,8 @@
 #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/sth_observer.h"
@@ -14,6 +16,8 @@ namespace net {
 
 namespace ct {
 
+struct SignedTreeHead;
+
 // A proxy for delegating new STH notifications to all registered
 // observers.
 // For each |observer| registered with RegisterObserver, the
@@ -28,10 +32,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).

[Ryan Sleevi, 2016/05/17 16:52:51]

Grammar:
Registers |observer| for new STH notifications. On registration, the |observer|
will be notified of the
latest STH for each log that the STHDistributor has observed.

[Eran Messeri, 2016/05/17 17:04:26]

Done.

   void RegisterObserver(STHObserver* observer) override;
+
+  // Unregisters |observer|, should only be called with observers
+  // that were previously registered  via RegisterObserver;

[Ryan Sleevi, 2016/05/17 16:52:51]

Style:
// Unregisters |observer|, which must have been previously registered via
RegisterObserver().

[Ryan Sleevi, 2016/05/17 16:52:51]

s/  / / (single space)

[Eran Messeri, 2016/05/17 17:04:26]

Done.

[Eran Messeri, 2016/05/17 17:04:26]

Done.

   void UnregisterObserver(STHObserver* observer) override;
 
  private:
+  // STHs from logs, one for each log.
+  std::vector<SignedTreeHead> observed_sths_;
+
+  // The observers for new STH notifications.
   base::ObserverList<STHObserver> observer_list_;
 };