Index: chrome/browser/chrome_content_browser_client.cc |
diff --git a/chrome/browser/chrome_content_browser_client.cc b/chrome/browser/chrome_content_browser_client.cc |
index 4f58d2fba0344ebf080b42057b37b3dc1c8a5d6a..c8e0ea2ab41b083c6fd074f182b712344cb99d61 100644 |
--- a/chrome/browser/chrome_content_browser_client.cc |
+++ b/chrome/browser/chrome_content_browser_client.cc |
@@ -1763,14 +1763,14 @@ void ChromeContentBrowserClient::AllowCertificateError( |
void ChromeContentBrowserClient::SelectClientCertificate( |
int render_process_id, |
int render_frame_id, |
+ const net::HttpNetworkSession* network_session, |
net::SSLCertRequestInfo* cert_request_info, |
const base::Callback<void(net::X509Certificate*)>& callback) { |
content::RenderFrameHost* rfh = content::RenderFrameHost::FromID( |
render_process_id, render_frame_id); |
WebContents* tab = WebContents::FromRenderFrameHost(rfh); |
if (!tab) { |
- // TODO(davidben): This makes the request hang, but returning no certificate |
- // also breaks. It should abort the request. See https://crbug.com/417092 |
+ NOTREACHED(); |
return; |
} |
@@ -1816,7 +1816,8 @@ void ChromeContentBrowserClient::SelectClientCertificate( |
} |
} |
- chrome::ShowSSLClientCertificateSelector(tab, cert_request_info, callback); |
+ chrome::ShowSSLClientCertificateSelector(tab, network_session, |
+ cert_request_info, callback); |
} |
void ChromeContentBrowserClient::AddCertificate( |