Fix CertLoader certificates_loaded() during notification.

chromeos/cert_loader.cc

Index: chromeos/cert_loader.cc
diff --git a/chromeos/cert_loader.cc b/chromeos/cert_loader.cc
index ff57d36c1c2ec8a0a4e161d39bbbba38afd69091..c71d6013ec4571c022bc0cc864ec6f29ef5a5267 100644
--- a/chromeos/cert_loader.cc
+++ b/chromeos/cert_loader.cc
@@ -44,8 +44,7 @@ base::TimeDelta GetNextRequestDelayMs(base::TimeDelta last_delay) {
 }
 
 void LoadNSSCertificates(net::CertificateList* cert_list) {
-  if (base::chromeos::IsRunningOnChromeOS())
-    net::NSSCertDatabase::GetInstance()->ListCerts(cert_list);
+  net::NSSCertDatabase::GetInstance()->ListCerts(cert_list);
 }
 
 void CallOpenPersistentNSSDB() {
@@ -345,8 +344,8 @@ void CertLoader::UpdateCertificates(net::CertificateList* cert_list) {
   // Ignore any existing certificates.
   cert_list_.swap(*cert_list);
 
-  NotifyCertificatesLoaded(!certificates_loaded_);
   certificates_loaded_ = true;
+  NotifyCertificatesLoaded(certificates_loaded_);

[stevenjb, 2013/07/26 19:40:53]

This will never call NotifyCertificatesLoaded with initial_load == true...

[pneubeck (no reviews), 2013/07/29 06:19:40]

Oh. totally missed that.

 
   certificates_update_running_ = false;
   if (certificates_update_required_)