Fixed issues with previous CL.

BUG=88782
TEST=None

Review URL: http://codereview.chromium.org/7335013

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92261 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 7 insertions, 9 deletions

diff --git a/net/base/origin_bound_cert_service.h b/net/base/origin_bound_cert_service.h
index 4502247..c1d65b9 100644
--- a/net/base/origin_bound_cert_service.h
+++ b/net/base/origin_bound_cert_service.h
@@ -8,30 +8,28 @@
 
 #include <string>
 
-#include "base/basictypes.h"
-#include "net/base/origin_bound_cert_store.h"
-#include "googleurl/src/gurl.h" // TODO(rkn): This feels wrong.
+class GURL;
 
 namespace net {
 
+class OriginBoundCertStore;
+
 // A class for creating and fetching origin bound certs.
 class OriginBoundCertService {
  public:
-
-  OriginBoundCertService(OriginBoundCertStore* origin_bound_cert_store)
+  explicit OriginBoundCertService(OriginBoundCertStore* origin_bound_cert_store)
       : origin_bound_cert_store_(origin_bound_cert_store) {}
 
   // TODO(rkn): Specify certificate type (RSA or DSA).
   // TODO(rkn): Key generation can be time consuming, so this should have an
   // asynchronous interface.
-  // This function will fetch the origin bound cert for the specified origin
-  // if one exists and it will create one otherwise.
+  // Fetches the origin bound cert for the specified origin if one exists
+  // and creates one otherwise. On success, |private_key_result| stores a
+  // PrivateKeyInfo struct, and |cert_result| stores a DER-encoded certificate.
   bool GetOriginBoundCert(const GURL& url,
                           std::string* private_key_result,
                           std::string* cert_result);
 
-  static std::string GetCertOriginFromURL(const GURL& url);
-
  private:
   OriginBoundCertStore* origin_bound_cert_store_;
 };