Don't use SSLStatus from FrameHostMsg_DidCommitProvisionalLoad and instead cache it on the browser …

content/browser/loader/navigation_resource_handler.h

Index: content/browser/loader/navigation_resource_handler.h
diff --git a/content/browser/loader/navigation_resource_handler.h b/content/browser/loader/navigation_resource_handler.h
index 48c86251d1648fb163cf49d109ed2368c78d27db..7262fd0eccdef3194770cf8361735fdcce868b16 100644
--- a/content/browser/loader/navigation_resource_handler.h
+++ b/content/browser/loader/navigation_resource_handler.h
@@ -10,7 +10,7 @@
 #include "content/browser/loader/stream_writer.h"
 
 namespace content {
-
+class CertStore;
 class NavigationURLLoaderImplCore;
 class ResourceDispatcherHostDelegate;
 
@@ -20,7 +20,8 @@ class NavigationResourceHandler : public ResourceHandler {
   NavigationResourceHandler(
       net::URLRequest* request,
       NavigationURLLoaderImplCore* core,
-      ResourceDispatcherHostDelegate* resource_dispatcher_host_delegate);
+      ResourceDispatcherHostDelegate* resource_dispatcher_host_delegate,
+      CertStore* cert_store);
   ~NavigationResourceHandler() override;
 
   // Called by the loader the cancel the request.
@@ -56,6 +57,7 @@ class NavigationResourceHandler : public ResourceHandler {
   NavigationURLLoaderImplCore* core_;
   StreamWriter writer_;
   ResourceDispatcherHostDelegate* resource_dispatcher_host_delegate_;
+  CertStore* cert_store_;

[clamy, 2016/08/26 00:06:57]

Why can't we use CertStore::GetInstance()?

[jam, 2016/08/26 03:36:43]

Like RDH and ResourceLoader, both support getting a mock cert store for tests
which is retrieved in RDHImpl::GetCertStore.

CertStore will be removed in followup patches.

 
   DISALLOW_COPY_AND_ASSIGN(NavigationResourceHandler);
 };