Fix webui cert viewer showing wrong cert chain on NSS and no chain on OpenSSL.

net/cert/x509_certificate.h

Index: net/cert/x509_certificate.h
diff --git a/net/cert/x509_certificate.h b/net/cert/x509_certificate.h
index 7aa48f068fa12867bd59beb9014dc078c60654ae..a6f8795133f22762f74cd17ee1e41b7631bd4de0 100644
--- a/net/cert/x509_certificate.h
+++ b/net/cert/x509_certificate.h
@@ -249,6 +249,12 @@ class NET_EXPORT X509Certificate
     return intermediate_ca_certs_;
   }
 
+  // Convenience method that fills |cert_chain| with the os_cert_handle()
+  // followed by GetIntermediateCertificates().  Ownership follows the "get"
+  // rule: it is the caller's responsibility to retain the elements of the
+  // result.
+  void GetCertificateChain(OSCertHandles* cert_chain) const;

[Ryan Sleevi, 2014/07/11 22:55:49]

I realize you added this based on msw's comments, but I'm not really a fan of
this.

X509Certificate exists as the canonical 'chain' type, and is meant to be what we
pass around for chains, because we rarely should be dealing in OSCertHandles.

[msw, 2014/07/11 23:01:15]

I'll defer to you, my concern was duplicated code, which yields maintenance
burdens and inconsistencies (some copies get fixed, some don't, etc.)...

+
 #if defined(OS_MACOSX)
   // Does this certificate's usage allow SSL client authentication?
   bool SupportsSSLClientAuth() const;