Allow browser tests to use a MockCertVerifier

net/cert/cert_verify_result.h

Index: net/cert/cert_verify_result.h
diff --git a/net/cert/cert_verify_result.h b/net/cert/cert_verify_result.h
index 9e6732552932b84e1d962becad0e508848d3c213..2ddb08ad0bb4f90ee6afee7a4a4e31315d9db77c 100644
--- a/net/cert/cert_verify_result.h
+++ b/net/cert/cert_verify_result.h
@@ -11,11 +11,10 @@
 #include "net/base/net_export.h"
 #include "net/cert/cert_status_flags.h"
 #include "net/cert/x509_cert_types.h"
+#include "net/cert/x509_certificate.h"

[estark, 2015/07/09 17:27:21]

I can't for the life of me figure out why this is needed. :( Without it,
CertVerifierBrowserTests fail to compile:

In file included from ../../chrome/test/base/cert_verifier_browser_test.cc:5:
In file included from ../../chrome/test/base/cert_verifier_browser_test.h:8:
In file included from ../../chrome/test/base/in_process_browser_test.h:10:
../../base/memory/ref_counted.h:398:6: error: member access into incomplete type
'net::X509Certificate'
  ptr->AddRef();
     ^
../../base/memory/ref_counted.h:316:7: note: in instantiation of member function
'scoped_refptr<net::X509Certificate>::AddRef' requested here
      AddRef(p);
      ^
../../base/memory/ref_counted.h:325:18: note: in instantiation of member
function 'scoped_refptr<net::X509Certificate>::operator=' requested here
    return *this = r.ptr_;
                 ^
../../net/cert/cert_verify_result.h:20:18: note: in instantiation of member
function 'scoped_refptr<net::X509Certificate>::operator=' requested here
class NET_EXPORT CertVerifyResult {
                 ^
../../net/cert/cert_verifier.h:20:7: note: forward declaration of
'net::X509Certificate'
class X509Certificate;

[Ryan Sleevi, 2015/07/09 19:40:26]

Because you're invoking a compiler-default copy-ctor, which triggers
AddRef/Release on line 37, which needs to be a complete type.

Was this error when you had additional code / tests using the CFBT? I'd expect
it when you return the CertVerifyResult in a .cc that hadn't included the
x509_certificate.h

 
 namespace net {
 
-class X509Certificate;
-
 // The result of certificate verification.
 class NET_EXPORT CertVerifyResult {
  public: