Add QuicStreamFactory as an observer of an SSLConfigService

net/quic/quic_stream_factory.cc

Index: net/quic/quic_stream_factory.cc
diff --git a/net/quic/quic_stream_factory.cc b/net/quic/quic_stream_factory.cc
index c011f6a9d5e6d1af82acbfd5bdb85ada4da14778..832cf909eb8a9ff21ccacd572ab8e25feb7eac82 100644
--- a/net/quic/quic_stream_factory.cc
+++ b/net/quic/quic_stream_factory.cc
@@ -596,6 +596,7 @@ QuicStreamRequest::CreateBidirectionalStreamImpl() {
 QuicStreamFactory::QuicStreamFactory(
     NetLog* net_log,
     HostResolver* host_resolver,
+    SSLConfigService* ssl_config_service,
     ClientSocketFactory* client_socket_factory,
     HttpServerProperties* http_server_properties,
     CertVerifier* cert_verifier,
@@ -690,7 +691,10 @@ QuicStreamFactory::QuicStreamFactory(
       num_push_streams_created_(0),
       status_(OPEN),
       task_runner_(nullptr),
+      ssl_config_service_(ssl_config_service),
       weak_factory_(this) {
+  if (ssl_config_service_.get())
+    ssl_config_service_->AddObserver(this);
   if (disable_quic_on_timeout_with_open_streams)
     threshold_timeouts_with_open_streams_ = 1;
   DCHECK(transport_security_state_);
@@ -748,6 +752,8 @@ QuicStreamFactory::~QuicStreamFactory() {
     STLDeleteElements(&(active_jobs_[server_id]));
     active_jobs_.erase(server_id);
   }
+  if (ssl_config_service_.get())
+    ssl_config_service_->RemoveObserver(this);

[Ryan Hamilton, 2016/06/29 23:16:15]

It'd sure be cool if we has some sort of ScopedObserver object that could be a
member of |this| which did AddObserver() when it was constructed and
RemoveObserver() when it was destroyed. But that's not something to do now.

   if (migrate_sessions_on_network_change_) {
     NetworkChangeNotifier::RemoveNetworkObserver(this);
   } else if (close_sessions_on_ip_change_) {