Collapse SSLPolicy/SSLPolicyBackend into SSLManager
SSLManager, SSLPolicy, and SSLPolicyBackend all pretty much ferry events
from one to the other, without any clear division of responsibility
among them. This CL collapses all the functionality into SSLManager for
clearer code.
BUG=558491
Review-Url: https://codereview.chromium.org/2395663002
Cr-Commit-Position: refs/heads/master@{#423240}
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index ec95918..081f049 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2986,7 +2986,8 @@
void WebContentsImpl::DidGetResourceResponseStart(
const ResourceRequestDetails& details) {
- controller_.ssl_manager()->DidStartResourceResponse(details);
+ controller_.ssl_manager()->DidStartResourceResponse(
+ details.url, details.has_certificate, details.ssl_cert_status);
FOR_EACH_OBSERVER(WebContentsObserver, observers_,
DidGetResourceResponseStart(details));
@@ -2994,8 +2995,6 @@
void WebContentsImpl::DidGetRedirectForResourceRequest(
const ResourceRedirectDetails& details) {
- controller_.ssl_manager()->DidReceiveResourceRedirect(details);
-
FOR_EACH_OBSERVER(
WebContentsObserver,
observers_,
