A bunch of interfaces: Return scoped_ptr<SSLCertificate>

webrtc/api/statscollector.cc

Index: webrtc/api/statscollector.cc
diff --git a/webrtc/api/statscollector.cc b/webrtc/api/statscollector.cc
index d77953b3bed1d49bb75a3be6a35c5d10f91c382b..ec176ef77ee581b73c79f85b7ba4b387ef649262 100644
--- a/webrtc/api/statscollector.cc
+++ b/webrtc/api/statscollector.cc
@@ -703,9 +703,9 @@ void StatsCollector::ExtractSessionInfo() {
         local_cert_report_id = r->id();
     }
 
-    rtc::scoped_ptr<rtc::SSLCertificate> cert;
-    if (pc_->session()->GetRemoteSSLCertificate(
-            transport_iter.second.transport_name, cert.accept())) {
+    auto cert = pc_->session()->GetRemoteSSLCertificate(

[torbjorng (webrtc), 2016/03/23 14:18:23]

I think this use of auto makes the code less readable; one needs to look for the
type instead of seeing it right away.

[kwiberg-webrtc, 2016/03/23 15:05:27]

Done. Also in a few other places; the only remaining autos that I introduce are
now in places where there wasn't a named variable at all in the old code.

+        transport_iter.second.transport_name);
+    if (cert) {
       StatsReport* r = AddCertificateReports(cert.get());
       if (r)
         remote_cert_report_id = r->id();