Certificate Transparency: Wire the TreeStateTracker up.

chrome/browser/io_thread.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_IO_THREAD_H_
 #define CHROME_BROWSER_IO_THREAD_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 30 matching lines @@
 }
 
 #if BUILDFLAG(ANDROID_JAVA_UI)
 namespace chrome {
 namespace android {
 class ExternalDataUseObserver;
 }
 }
 #endif  // BUILDFLAG(ANDROID_JAVA_UI)
 
+namespace certificate_transparency {
+class TreeStateTracker;
+}
+
 namespace chrome_browser_net {
 class DnsProbeService;
 }
 
 namespace data_usage {
 class DataUseAggregator;
 }
 
 namespace extensions {
 class EventRouterForwarder;
@@ skipping 17 matching lines @@
 class NetworkDelegate;
 class NetworkQualityEstimator;
 class ProxyConfigService;
 class ProxyService;
 class SSLConfigService;
 class TransportSecurityState;
 class URLRequestBackoffManager;
 class URLRequestContext;
 class URLRequestContextGetter;
 class URLRequestJobFactory;
+
+namespace ct {
+class STHObserver;
+}
+
 }  // namespace net
 
 namespace net_log {
 class ChromeNetLog;
 }
 
 namespace policy {
 class PolicyService;
 }  // namespace policy
 
@@ skipping 320 matching lines @@
   // called on the IO thread.
   void ClearHostCache();
 
   const net::HttpNetworkSession::Params& NetworkSessionParams() const;
 
   base::TimeTicks creation_time() const;
 
   // 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().
   friend class SystemURLRequestContextGetter;
 
   friend class test::IOThreadPeer;
 
   // BrowserThreadDelegate implementation, runs on the IO thread.
   // This handles initialization and destruction of state that must
   // live on the IO thread.
@@ skipping 65 matching lines @@
   // lifetime of the IO thread.
 
   Globals* globals_;
 
   net::HttpNetworkSession::Params params_;
   NetworkSessionConfigurator network_session_configurator_;
 
   // 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_;
 
   BooleanPrefMember quick_check_enabled_;
 
   // Store HTTP Auth-related policies in this thread.
   // TODO(aberent) Make the list of auth schemes a PrefMember, so that the
   // policy can change after startup (https://crbug/549273).
   std::string auth_schemes_;
@@ skipping 37 matching lines @@
   // Callback for updating data use prefs which needs to be initialized on UI
   // thread and passed to |ChromeNetworkDelegate|.
   metrics::UpdateUsagePrefCallbackType metrics_data_use_forwarder_;
 
   base::WeakPtrFactory<IOThread> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(IOThread);
 };
 
 #endif  // CHROME_BROWSER_IO_THREAD_H_