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

Unified Diff: net/android/network_library.h

Issue 10171009: Update net/android/network_library.cc with fresher code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: added comments, fixed caller Created 8 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
Index: net/android/network_library.h
diff --git a/net/android/network_library.h b/net/android/network_library.h
index 0694de60072baae6ac889b0aa2fb8b04520346cb..98c7b9d668c80022035f980a1024ab83754e0d00 100644
--- a/net/android/network_library.h
+++ b/net/android/network_library.h
@@ -19,8 +19,6 @@ namespace android {
enum VerifyResult {
// Certificate verification was successful.
VERIFY_OK,
- // Certificate domain name doesn't match host name.
- VERIFY_BAD_HOSTNAME,
// Certificate verification was failed. There is no detail error information
// given by Android API.
VERIFY_NO_TRUSTED_ROOT,
@@ -30,11 +28,9 @@ enum VerifyResult {
// |cert_chain| is DER encoded chain of certificates, with the server's own
// certificate listed first.
-// |hostname| is validated against the supplied cert. |auth_type| is as per
-// the Java X509Certificate.checkServerTrusted method.
+// |auth_type| is as per the Java X509Certificate.checkServerTrusted method.
VerifyResult VerifyX509CertChain(const std::vector<std::string>& cert_chain,
- const std::string& hostname,
const std::string& auth_type);
// Helper for the <keygen> handler. Passes the DER-encoded key pair via

Powered by Google App Engine
This is Rietveld 408576698