Certificate Transparency: Wire the TreeStateTracker up.

chrome/browser/io_thread.h

Index: chrome/browser/io_thread.h
diff --git a/chrome/browser/io_thread.h b/chrome/browser/io_thread.h
index 85210ba1abdb46ce0de8348ce584a049ddf91364..7b6fb746a225bd25c7874c550224e11ac3e6ac63 100644
--- a/chrome/browser/io_thread.h
+++ b/chrome/browser/io_thread.h
@@ -48,6 +48,10 @@ class ExternalDataUseObserver;
 }
 #endif  // BUILDFLAG(ANDROID_JAVA_UI)
 
+namespace certificate_transparency {
+class TreeStateTracker;
+}
+
 namespace chrome_browser_net {
 class DnsProbeService;
 }
@@ -85,6 +89,11 @@ class URLRequestBackoffManager;
 class URLRequestContext;
 class URLRequestContextGetter;
 class URLRequestJobFactory;
+
+namespace ct {
+class STHObserver;
+}
+
 }  // namespace net
 
 namespace net_log {
@@ -425,6 +434,12 @@ class IOThread : public content::BrowserThreadDelegate {
   // Returns the callback for updating data use prefs.
   const metrics::UpdateUsagePrefCallbackType& GetMetricsDataUseForwarder();
 
+  // Registers the |observer| for new STH notifications.
+  void RegisterSTHObserver(net::ct::STHObserver* observer);

[Sorin Jianu, 2016/05/25 16:17:48]

Can we do more smart pointers instead of raw pointers?

[Eran Messeri, 2016/05/25 16:26:18]

I couldn't see a way.
I don't think it is necessary because the ownership of the observer is not
passed into RegisterSTHObserver - it remains with whichever object created the
observer.
Per
https://groups.google.com/a/chromium.org/forum/#!msg/chromium-dev/TlL1D-Djta0...,
since ownership is not passed, I am passing a raw pointer.

In this particular case, each entity that registers an STH Observer (the
IOThread or ProfileIOData) is responsible for un-registering it before
destroying it, as the same object would be registered in a local context with
the CTVerifier object held by the same entity.

+
+  // Un-registers the |observer|.
+  void UnregisterSTHObserver(net::ct::STHObserver* observer);
+
  private:
   // Provide SystemURLRequestContextGetter with access to
   // InitSystemRequestContext().
@@ -510,6 +525,8 @@ class IOThread : public content::BrowserThreadDelegate {
   // Observer that logs network changes to the ChromeNetLog.
   std::unique_ptr<net::LoggingNetworkChangeObserver> network_change_observer_;
 
+  std::unique_ptr<certificate_transparency::TreeStateTracker> ct_tree_tracker_;
+
   BooleanPrefMember system_enable_referrers_;
 
   BooleanPrefMember dns_client_enabled_;