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 2b9c6b9636a062d88e51554bbd59cd5f6185196e..d40df82a00a55427bdc0c16ca17e144796138dc7 100644 |
--- a/chrome/browser/chrome_content_browser_client.cc |
+++ b/chrome/browser/chrome_content_browser_client.cc |
@@ -1739,14 +1739,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) { |
- NOTREACHED(); |
+ // TODO(davidben): This makes the request hang, but returning no certificate |
+ // also breaks. It should abort the request. See https://crbug.com/417092 |
return; |
} |
@@ -1792,8 +1792,7 @@ void ChromeContentBrowserClient::SelectClientCertificate( |
} |
} |
- chrome::ShowSSLClientCertificateSelector(tab, network_session, |
- cert_request_info, callback); |
+ chrome::ShowSSLClientCertificateSelector(tab, cert_request_info, callback); |
} |
void ChromeContentBrowserClient::AddCertificate( |