Handle cases when user cert database has NULL slots

chrome/browser/certificate_manager_model.h

Index: chrome/browser/certificate_manager_model.h
diff --git a/chrome/browser/certificate_manager_model.h b/chrome/browser/certificate_manager_model.h
index 7364b32714bbf2727e32319515812af7c7d89866..0a9783724fa49f2808351b3a65928cd5ac01b62d 100644
--- a/chrome/browser/certificate_manager_model.h
+++ b/chrome/browser/certificate_manager_model.h
@@ -56,6 +56,7 @@ class CertificateManagerModel {
 
   ~CertificateManagerModel();
 
+  bool is_user_db_available() const { return is_user_db_available_; }
   bool is_tpm_available() const { return is_tpm_available_; }
 
   // Accessor for read-only access to the underlying NSSCertDatabase.
@@ -124,6 +125,7 @@ class CertificateManagerModel {
 
  private:
   CertificateManagerModel(net::NSSCertDatabase* nss_cert_database,
+                          bool is_user_db_available,
                           bool is_tpm_available,
                           Observer* observer);
 
@@ -131,6 +133,7 @@ class CertificateManagerModel {
   // file for details.
   static void DidGetCertDBOnUIThread(
       net::NSSCertDatabase* cert_db,
+      bool is_user_db_available,
       bool is_tpm_available,
       CertificateManagerModel::Observer* observer,
       const CreationCallback& callback);
@@ -148,6 +151,7 @@ class CertificateManagerModel {
 
   net::NSSCertDatabase* cert_db_;
   net::CertificateList cert_list_;
+  bool is_user_db_available_;

[James Hawkins, 2014/03/17 18:58:22]

It's not clear to me from the variable name exactly what this member variable
is; please add documentation.

[tbarzic, 2014/03/17 19:17:38]

Done, hope it's clearer now.

   bool is_tpm_available_;
 
   // The observer to notify when certificate list is refreshed.