Net-related fixups for scoped_refptr conversion operator cleanup.

chrome/browser/ui/views/ssl_client_certificate_selector.cc

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 4a96ca97f5632bae9e74641b8a0461e85b907f3c..2e313449ff5a651c578a2355120efea76b4b1ea5 100644
--- a/chrome/browser/ui/views/ssl_client_certificate_selector.cc
+++ b/chrome/browser/ui/views/ssl_client_certificate_selector.cc
@@ -83,10 +83,10 @@ void CertificateSelectorTableModel::SetObserver(
 SSLClientCertificateSelector::SSLClientCertificateSelector(
     content::WebContents* web_contents,
     const net::HttpNetworkSession* network_session,
-    net::SSLCertRequestInfo* cert_request_info,
+    const scoped_refptr<net::SSLCertRequestInfo>& cert_request_info,
     const chrome::SelectCertificateCallback& callback)
     : SSLClientAuthObserver(network_session, cert_request_info, callback),
-      model_(new CertificateSelectorTableModel(cert_request_info)),
+      model_(new CertificateSelectorTableModel(cert_request_info.get())),

[mmenke, 2014/08/28 13:47:32]

We aren't holding onto a pointer to cert_request_info either in
SSLClientCertificateSelector or CertificateSelectorTableModel...  If it's worth
modifying SSLClientCertificateSelector's constructor in this case, we should
modify CertificateSelectorTableModel's constructor, too, no?

[dcheng, 2014/08/28 16:23:15]

This class doesn't, but its base class (SSLClientAuthObserver) refs it.
CertificateSelectorTableModel doesn't ref it at all, as far as I can tell.

       web_contents_(web_contents),
       table_(NULL),
       view_cert_button_(NULL) {