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

content/browser/loader/navigation_resource_handler.h

 // Copyright 2014 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 CONTENT_BROWSER_LOADER_NAVIGATION_RESOURCE_HANDLER_H_
 #define CONTENT_BROWSER_LOADER_NAVIGATION_RESOURCE_HANDLER_H_
 
 #include "base/macros.h"
 #include "content/browser/loader/resource_handler.h"
 #include "content/browser/loader/stream_writer.h"
 
 namespace content {
-
+class CertStore;
 class NavigationURLLoaderImplCore;
 class ResourceDispatcherHostDelegate;
 
 // PlzNavigate: The leaf ResourceHandler used with NavigationURLLoaderImplCore.
 class NavigationResourceHandler : public ResourceHandler {
  public:
   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.
   void Cancel();
 
   // Called to the loader to resume a paused redirect.
   void FollowRedirect();
 
   // Called to proceed with the response.
   void ProceedWithResponse();
@@ skipping 15 matching lines @@
   void OnDataDownloaded(int bytes_downloaded) override;
 
  private:
   // Clears |core_| and its reference to the resource handler. After calling
   // this, the lifetime of the request is no longer tied to |core_|.
   void DetachFromCore();
 
   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);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_NAVIGATION_RESOURCE_HANDLER_H_