Separate CertVerifyProcAndroid from CertVerifyProcOpenSSL

net/base/cert_verify_proc_android.h

 // 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.
 
-#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,
                              const std::string& hostname,
                              int flags,
                              CRLSet* crl_set,
                              CertVerifyResult* verify_result) OVERRIDE;
 };
 
 }  // namespace net
 
-#endif  // NET_BASE_CERT_VERIFY_PROC_WIN_H_
+#endif  // NET_BASE_CERT_VERIFY_PROC_ANDROID_H_