Fix Android build following r128172

Fix Android build following r128172

BUG=114343
TEST=goes green
TBR=wtc

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=128174

[Ryan Sleevi, 2012-03-22 07:51:12]

TBR wtc.

jnd as FYI in case this breaks anything while syncing.

[wtc, 2012-03-22 17:43:13]

http://codereview.chromium.org/9836001/diff/1/net/base/cert_verify_proc_opens...
File net/base/cert_verify_proc_openssl.cc (right):

http://codereview.chromium.org/9836001/diff/1/net/base/cert_verify_proc_opens...
net/base/cert_verify_proc_openssl.cc:232: GetChainDEREncodedBytes(cert,
&cert_bytes);

Should this be
    cert->GetChainDEREncodedBytes(&cert_bytes);
?

[Ryan Sleevi, 2012-03-22 17:45:58]

On 2012/03/22 17:43:13, wtc wrote:
>
http://codereview.chromium.org/9836001/diff/1/net/base/cert_verify_proc_opens...
> File net/base/cert_verify_proc_openssl.cc (right):
> 
>
http://codereview.chromium.org/9836001/diff/1/net/base/cert_verify_proc_opens...
> net/base/cert_verify_proc_openssl.cc:232: GetChainDEREncodedBytes(cert,
> &cert_bytes);
> 
> Should this be
>     cert->GetChainDEREncodedBytes(&cert_bytes);
> ?

No.

GetChainDEREncodedBytes was moved into a static method within this file, removed
from x509_certificate.h / x509_certificate_openssl.cc.

GetChainDEREncodedBytes is then implemented on top of the "common"
X509Certificate::GetDEREncoded(OSCertHandle), and the static method is
responsible for building the chain.

The goal being two-fold: Remove the OS-specific bits from x509_certificate.h,
and move the single usage of this method into its corresponding implementation.

[wtc, 2012-03-22 17:46:53]

Thank you.  LGTM.