Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5723)

Unified Diff: chrome/browser/ui/views/ssl_client_certificate_selector.cc

Issue 2838243002: Remove client_certs from SSLCertRequestInfo. (Closed)
Patch Set: revert stray whitespace change Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/ssl_client_certificate_selector.cc
diff --git a/chrome/browser/ui/views/ssl_client_certificate_selector.cc b/chrome/browser/ui/views/ssl_client_certificate_selector.cc
index 0830efb9e2c4ab2d371cb65f83c816c87997bf43..f30964ca2b3776c2b2a35b9345f5bc889e6bcace 100644
--- a/chrome/browser/ui/views/ssl_client_certificate_selector.cc
+++ b/chrome/browser/ui/views/ssl_client_certificate_selector.cc
@@ -27,8 +27,9 @@
SSLClientCertificateSelector::SSLClientCertificateSelector(
content::WebContents* web_contents,
const scoped_refptr<net::SSLCertRequestInfo>& cert_request_info,
+ net::CertificateList client_certs,
std::unique_ptr<content::ClientCertificateDelegate> delegate)
- : CertificateSelector(cert_request_info->client_certs, web_contents),
+ : CertificateSelector(std::move(client_certs), web_contents),
SSLClientAuthObserver(web_contents->GetBrowserContext(),
cert_request_info,
std::move(delegate)),
@@ -99,6 +100,7 @@ namespace chrome {
void ShowSSLClientCertificateSelector(
content::WebContents* contents,
net::SSLCertRequestInfo* cert_request_info,
+ net::CertificateList client_certs,
std::unique_ptr<content::ClientCertificateDelegate> delegate) {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
@@ -110,7 +112,8 @@ void ShowSSLClientCertificateSelector(
return;
SSLClientCertificateSelector* selector = new SSLClientCertificateSelector(
- contents, cert_request_info, std::move(delegate));
+ contents, cert_request_info, std::move(client_certs),
+ std::move(delegate));
selector->Init();
selector->Show();
}

Powered by Google App Engine
This is Rietveld 408576698