Omnibox M5 work, part 1: Security changes...

chrome/browser/cert_store.h

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2010 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.
 
 #ifndef CHROME_BROWSER_CERT_STORE_H_
 #define CHROME_BROWSER_CERT_STORE_H_
 
 #include <map>
 
 #include "base/lock.h"
 #include "base/singleton.h"
@@ skipping 15 matching lines @@
   // Returns the singleton instance of the CertStore.
   static CertStore* GetSharedInstance();
 
   // Stores the specified cert and returns the id associated with it.  The cert
   // is associated to the specified RenderProcessHost.
   // When all the RenderProcessHosts associated with a cert have exited, the
   // cert is removed from the store.
   // Note: ids starts at 1.
   int StoreCert(net::X509Certificate* cert, int render_process_host_id);
 
-  // Retrieves the previously stored cert associated with the specified
-  // |cert_id| and set it in |cert|.  Returns false if no cert was found for
-  // that id.
+  // Tries to retrieve the previously stored cert associated with the specified
+  // |cert_id|.  Returns whether the cert could be found, and, if |cert| is
+  // non-NULL, copies it in.
   bool RetrieveCert(int cert_id, scoped_refptr<net::X509Certificate>* cert);
 
   // NotificationObserver implementation.
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
  private:
   friend struct DefaultSingletonTraits<CertStore>;
 
@@ skipping 23 matching lines @@
   int next_cert_id_;
 
   // This lock protects: process_to_ids_, id_to_processes_, id_to_cert_ and
   //                     cert_to_id_.
   Lock cert_lock_;
 
   DISALLOW_COPY_AND_ASSIGN(CertStore);
 };
 
 #endif  // CHROME_BROWSER_CERT_STORE_H_