Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(75)

Unified Diff: net/base/cert_verify_proc.cc

Issue 9584041: Create stubs for system certificate validation. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix across platforms Created 8 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/cert_verify_proc.cc
diff --git a/net/base/cert_verify_proc.cc b/net/base/cert_verify_proc.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4a1c94d77bd248ce4502285115a83cc946614248
--- /dev/null
+++ b/net/base/cert_verify_proc.cc
@@ -0,0 +1,42 @@
+// Copyright (c) 2012 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/cert_verify_proc.h"
+
+#include "build/build_config.h"
+#include "net/base/cert_verify_proc_mac.h"
+#include "net/base/cert_verify_proc_nss.h"
+#include "net/base/cert_verify_proc_openssl.h"
+#include "net/base/cert_verify_proc_win.h"
wtc 2012/03/13 22:37:57 Nit: it seems that these platform-specific headers
+
+namespace net {
+
+// static
+CertVerifyProc* CertVerifyProc::CreateDefault() {
+#if defined(USE_NSS)
+ return new CertVerifyProcNSS();
+#elif defined(USE_OPENSSL)
+ return new CertVerifyProcOpenSSL();
+#elif defined(OS_MACOSX)
+ return new CertVerifyProcMac();
+#elif defined(OS_WIN)
+ return new CertVerifyProcWin();
+#else
+#error Implement certificate verification for your platform
+#endif
+}
+
+CertVerifyProc::CertVerifyProc() {}
+
+CertVerifyProc::~CertVerifyProc() {}
+
+int CertVerifyProc::Verify(X509Certificate* cert,
+ const std::string& hostname,
+ int flags,
+ CRLSet* crl_set,
+ CertVerifyResult* verify_result) {
+ return VerifyInternal(cert, hostname, flags, crl_set, verify_result);
wtc 2012/03/13 22:37:57 Nit: the purpose of the Verify/VerifyInternal spli
+}
+
+} // namespace net

Powered by Google App Engine
This is Rietveld 408576698