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

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

Issue 16998003: Update CrOS to use scoped_refptr<T>::get() rather than implicit "operator T*" (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased Created 7 years, 6 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 cbc2b0adb7b9b4e832fb8a353ec848e338c4a0a5..4da3e799bf9867431a78881a51a0e8b0f95d7d16 100644
--- a/chrome/browser/ui/views/ssl_client_certificate_selector.cc
+++ b/chrome/browser/ui/views/ssl_client_certificate_selector.cc
@@ -64,7 +64,7 @@ class CertificateSelectorTableModel : public ui::TableModel {
CertificateSelectorTableModel::CertificateSelectorTableModel(
net::SSLCertRequestInfo* cert_request_info) {
for (size_t i = 0; i < cert_request_info->client_certs.size(); ++i) {
- net::X509Certificate* cert = cert_request_info->client_certs[i];
+ net::X509Certificate* cert = cert_request_info->client_certs[i].get();
string16 text = l10n_util::GetStringFUTF16(
IDS_CERT_SELECTOR_TABLE_CERT_FORMAT,
UTF8ToUTF16(cert->subject().GetDisplayName()),
@@ -161,7 +161,7 @@ net::X509Certificate* SSLClientCertificateSelector::GetSelectedCert() const {
if (selected >= 0 &&
selected < static_cast<int>(
cert_request_info()->client_certs.size()))
- return cert_request_info()->client_certs[selected];
+ return cert_request_info()->client_certs[selected].get();
return NULL;
}
« no previous file with comments | « chrome/browser/ui/views/simple_message_box_views.cc ('k') | chrome/browser/ui/webui/certificate_viewer_webui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698