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 2ff880459fb57c0fa2ed83c445ca3e7b80e6ae81..d495833c689438789799f322d8a40bf85eed230f 100644 |
--- a/chrome/browser/chrome_content_browser_client.cc |
+++ b/chrome/browser/chrome_content_browser_client.cc |
@@ -1725,14 +1725,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; |
} |
@@ -1778,8 +1778,7 @@ void ChromeContentBrowserClient::SelectClientCertificate( |
} |
} |
- chrome::ShowSSLClientCertificateSelector(tab, network_session, |
- cert_request_info, callback); |
+ chrome::ShowSSLClientCertificateSelector(tab, cert_request_info, callback); |
} |
void ChromeContentBrowserClient::AddCertificate( |