Separate CertVerifyProcAndroid from CertVerifyProcOpenSSL

net/base/cert_verify_proc_android.h

Index: net/base/cert_verify_proc_android.h
diff --git a/net/base/cert_verify_proc_win.h b/net/base/cert_verify_proc_android.h
similarity index 63%
copy from net/base/cert_verify_proc_win.h
copy to net/base/cert_verify_proc_android.h
index e0307141ab393216273b15996cb04c72f70fe893..acc97d9ce9e83a144c2756fe68c18116b051a531 100644
--- a/net/base/cert_verify_proc_win.h
+++ b/net/base/cert_verify_proc_android.h
@@ -2,21 +2,20 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef NET_BASE_CERT_VERIFY_PROC_WIN_H_
-#define NET_BASE_CERT_VERIFY_PROC_WIN_H_
+#ifndef NET_BASE_CERT_VERIFY_PROC_ANDROID_H_
+#define NET_BASE_CERT_VERIFY_PROC_ANDROID_H_
 
 #include "net/base/cert_verify_proc.h"
 
 namespace net {
 
-// Performs certificate path construction and validation using Windows'
-// CryptoAPI.
-class CertVerifyProcWin : public CertVerifyProc {
+// Performs platform-based certificate verification on Android.

[Ryan Sleevi, 2012/12/13 02:16:36]

This comment is not as clear as what we've tried for the other platforms.

You should mention JNI here

// Performs certificate verification on Android by using JNI to [whatever the
API/framework is called]

[ppi, 2012/12/13 04:49:09]

Absolutely, addressed in patch set 2.

+class CertVerifyProcAndroid : public CertVerifyProc {
  public:
-  CertVerifyProcWin();
+  CertVerifyProcAndroid();
 
  protected:
-  virtual ~CertVerifyProcWin();
+  virtual ~CertVerifyProcAndroid();
 
  private:
   virtual int VerifyInternal(X509Certificate* cert,
@@ -28,4 +27,4 @@ class CertVerifyProcWin : public CertVerifyProc {
 
 }  // namespace net
 
-#endif  // NET_BASE_CERT_VERIFY_PROC_WIN_H_
+#endif  // NET_BASE_CERT_VERIFY_PROC_ANDROID_H_