Index: chrome/browser/chromeos/network_login_observer.cc |
diff --git a/chrome/browser/chromeos/network_login_observer.cc b/chrome/browser/chromeos/network_login_observer.cc |
index 3511865dcd630370482fc32f53513e353ec57655..71c22fdd926dc484cd3057f57446852b4c4dd2ad 100644 |
--- a/chrome/browser/chromeos/network_login_observer.cc |
+++ b/chrome/browser/chromeos/network_login_observer.cc |
@@ -4,7 +4,6 @@ |
#include "chrome/browser/chromeos/network_login_observer.h" |
-#include "chrome/browser/chromeos/cros/cros_library.h" |
#include "chrome/browser/chromeos/cros/network_library.h" |
#include "chrome/browser/chromeos/options/network_config_view.h" |
#include "chromeos/network/network_state_handler.h" |
@@ -14,11 +13,16 @@ |
namespace chromeos { |
NetworkLoginObserver::NetworkLoginObserver() { |
- CrosLibrary::Get()->GetCertLibrary()->AddObserver(this); |
+ // CertLoader does not get initialized in many unit tests even though |
+ // NetworkLibrary which owns this class does. TODO(stevenjb): Eliminate |
+ // this class along with NetworkLibrary, crbug.com/154852. |
+ if (CertLoader::IsInitialized()) |
+ CertLoader::Get()->AddObserver(this); |
} |
NetworkLoginObserver::~NetworkLoginObserver() { |
- CrosLibrary::Get()->GetCertLibrary()->RemoveObserver(this); |
+ if (CertLoader::IsInitialized()) |
+ CertLoader::Get()->RemoveObserver(this); |
} |
void NetworkLoginObserver::OnNetworkManagerChanged(NetworkLibrary* cros) { |
@@ -84,7 +88,9 @@ void NetworkLoginObserver::OnNetworkManagerChanged(NetworkLibrary* cros) { |
} |
} |
-void NetworkLoginObserver::OnCertificatesLoaded(bool initial_load) { |
+void NetworkLoginObserver::OnCertificatesLoaded( |
+ const net::CertificateList& cert_list, |
+ bool initial_load) { |
if (initial_load) { |
// Once certificates have loaded, connect to the "best" available network. |
NetworkStateHandler::Get()->ConnectToBestWifiNetwork(); |