Cleanup unreachable cert adding code

net/socket/client_socket_factory.cc

 // 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.
 
 #include "net/socket/client_socket_factory.h"
 
 #include <utility>
 
 #include "base/lazy_instance.h"
 #include "build/build_config.h"
@@ skipping 15 matching lines @@
   DefaultClientSocketFactory() {
     CertDatabase::GetInstance()->AddObserver(this);
   }
 
   ~DefaultClientSocketFactory() override {
     // Note: This code never runs, as the factory is defined as a Leaky
     // singleton.
     CertDatabase::GetInstance()->RemoveObserver(this);
   }
 
-  void OnCertAdded(const X509Certificate* cert) override {
+  void OnCertDBChanged(const X509Certificate* cert) override {
+    // Flush sockets whenever CA trust changes.
     ClearSSLSessionCache();
   }
 
-  void OnCACertChanged(const X509Certificate* cert) override {
-    // Per wtc, we actually only need to flush when trust is reduced.
-    // Always flush now because OnCACertChanged does not tell us this.
-    // See comments in ClientSocketPoolManager::OnCACertChanged.
-    ClearSSLSessionCache();
-  }
-
   std::unique_ptr<DatagramClientSocket> CreateDatagramClientSocket(
       DatagramSocket::BindType bind_type,
       const RandIntCallback& rand_int_cb,
       NetLog* net_log,
       const NetLogSource& source) override {
     return std::unique_ptr<DatagramClientSocket>(
         new UDPClientSocket(bind_type, rand_int_cb, net_log, source));
   }
 
   std::unique_ptr<StreamSocket> CreateTransportClientSocket(
@@ skipping 21 matching lines @@
     g_default_client_socket_factory = LAZY_INSTANCE_INITIALIZER;
 
 }  // namespace
 
 // static
 ClientSocketFactory* ClientSocketFactory::GetDefaultFactory() {
   return g_default_client_socket_factory.Pointer();
 }
 
 }  // namespace net