Remove members from NavigationHandleImpl 4
This CL is part of a series of CLs that removes members from
NavigationHandleImpl and queries them from NavigationRequest instead.
This CL removes parameters passed to NavigationHandleImpl::WillProcessResponse.
Bug: 916537
Change-Id: If35835e7034238fabe8dc4b9a4d933093b1249e1
Reviewed-on: https://chromium-review.googlesource.com/c/1450193
Reviewed-by: Alex Moshchuk <[email protected]>
Reviewed-by: Carlos IL <[email protected]>
Commit-Queue: Camille Lamy <[email protected]>
Cr-Commit-Position: refs/heads/master@{#635443}diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 69498d13..79cc8128 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -4284,10 +4284,14 @@
// SSLInfo is not needed on subframe navigations since the main-frame
// certificate is the only one that can be inspected (using the info
// bubble) without refreshing the page with DevTools open.
- if (navigation_handle->IsInMainFrame())
+ if (navigation_handle->IsInMainFrame()) {
controller_.ssl_manager()->DidStartResourceResponse(
navigation_handle->GetURL(),
- net::IsCertStatusError(navigation_handle->GetSSLInfo().cert_status));
+ navigation_handle->GetSSLInfo().has_value()
+ ? net::IsCertStatusError(
+ navigation_handle->GetSSLInfo()->cert_status)
+ : false);
+ }
SetNotWaitingForResponse();
}
