Add functionality to OriginBoundCertStore interface and implementations.

net/base/default_origin_bound_cert_store.cc

 // Copyright (c) 2011 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/base/default_origin_bound_cert_store.h"
 
 #include "base/message_loop.h"
 
 namespace net {
 
@@ skipping 40 matching lines @@
   base::AutoLock autolock(lock_);
   InitIfNecessary();
 
   InternalDeleteOriginBoundCert(origin);
   InternalInsertOriginBoundCert(origin,
                                 new OriginBoundCert(origin, private_key, cert));
 
   return true;
 }
 
+void DefaultOriginBoundCertStore::DeleteOriginBoundCert(
+    const std::string& origin) {
+  base::AutoLock autolock(lock_);
+  InitIfNecessary();
+  InternalDeleteOriginBoundCert(origin);
+}
+
+void DefaultOriginBoundCertStore::DeleteAll() {
+  base::AutoLock autolock(lock_);
+  InitIfNecessary();
+  for (OriginBoundCertMap::iterator it = origin_bound_certs_.begin();
+       it != origin_bound_certs_.end(); ++it) {
+    InternalDeleteOriginBoundCert(it->second->origin());
+  }
+}
+
+void DefaultOriginBoundCertStore::GetAllOriginBoundCerts(
+    std::vector<OriginBoundCertInfo>* origin_bound_certs) {
+  base::AutoLock autolock(lock_);
+  InitIfNecessary();
+  for (OriginBoundCertMap::iterator it = origin_bound_certs_.begin();
+       it != origin_bound_certs_.end(); ++it) {
+    struct OriginBoundCertInfo cert_info = {

[wtc, 2011/08/10 21:35:59]

Remove "struct".

We should be able to just do
  origin_bound_certs->push_back(*it->second);
You should add a TODO comment about this.

+      it->second->origin(),
+      it->second->private_key(),
+      it->second->cert()
+    };
+    origin_bound_certs->push_back(cert_info);
+  }
+}
+
 int DefaultOriginBoundCertStore::GetCertCount() {
   base::AutoLock autolock(lock_);
   InitIfNecessary();
 
   return origin_bound_certs_.size();
 }
 
 DefaultOriginBoundCertStore::~DefaultOriginBoundCertStore() {
-  DeleteAll();
+  DeleteAllInMemory();
 }
 
-void DefaultOriginBoundCertStore::DeleteAll() {
+void DefaultOriginBoundCertStore::DeleteAllInMemory() {
   base::AutoLock autolock(lock_);
 
   for (OriginBoundCertMap::iterator it = origin_bound_certs_.begin();
-      it != origin_bound_certs_.end(); it++) {
+       it != origin_bound_certs_.end(); ++it) {
     delete it->second;
   }
   origin_bound_certs_.clear();
 }
 
 void DefaultOriginBoundCertStore::InitStore() {
   lock_.AssertAcquired();
 
   DCHECK(store_) << "Store must exist to initialize";
 
@@ skipping 41 matching lines @@
     const std::string& origin,
     const std::string& private_key,
     const std::string& cert)
     : origin_(origin),
       private_key_(private_key),
       cert_(cert) {}
 
 DefaultOriginBoundCertStore::PersistentStore::PersistentStore() {}
 
 }  // namespace net