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

Unified Diff: net/cert/cert_verify_proc.cc

Issue 2862543003: Remove CertVerifyProcOpenSSL. (Closed)
Patch Set: . Created 3 years, 8 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
« no previous file with comments | « net/BUILD.gn ('k') | net/cert/cert_verify_proc_openssl.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/cert/cert_verify_proc.cc
diff --git a/net/cert/cert_verify_proc.cc b/net/cert/cert_verify_proc.cc
index a2a8a9e61a2fba42c2bb5fc42b76cd6d6d1349e8..13f037555bc89fb9fde20ec0db4c134790d5925f 100644
--- a/net/cert/cert_verify_proc.cc
+++ b/net/cert/cert_verify_proc.cc
@@ -33,8 +33,6 @@
#if defined(USE_NSS_CERTS)
#include "net/cert/cert_verify_proc_nss.h"
-#elif defined(USE_OPENSSL_CERTS) && !defined(OS_ANDROID)
-#include "net/cert/cert_verify_proc_openssl.h"
#elif defined(OS_ANDROID)
#include "net/cert/cert_verify_proc_android.h"
#elif defined(OS_IOS)
@@ -504,8 +502,6 @@ WARN_UNUSED_RESULT bool InspectSignatureAlgorithmsInChain(
CertVerifyProc* CertVerifyProc::CreateDefault() {
#if defined(USE_NSS_CERTS)
return new CertVerifyProcNSS();
-#elif defined(USE_OPENSSL_CERTS) && !defined(OS_ANDROID)
- return new CertVerifyProcOpenSSL();
#elif defined(OS_ANDROID)
return new CertVerifyProcAndroid();
#elif defined(OS_IOS)
@@ -515,7 +511,7 @@ CertVerifyProc* CertVerifyProc::CreateDefault() {
#elif defined(OS_WIN)
return new CertVerifyProcWin();
#else
- return NULL;
+#error Unsupported platform
davidben 2017/05/03 18:54:48 No caller survives this being NULL. Better to have
#endif
}
« no previous file with comments | « net/BUILD.gn ('k') | net/cert/cert_verify_proc_openssl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698