RTCCertificates added to RTCConfiguration, used by WebRtcSession/-DescriptionFactory

talk/app/webrtc/peerconnectioninterface.h

Index: talk/app/webrtc/peerconnectioninterface.h
diff --git a/talk/app/webrtc/peerconnectioninterface.h b/talk/app/webrtc/peerconnectioninterface.h
index ac6280fc24b54045e5f9f8750b3fcddff952ffcb..a98f57ad69b98cc45097ba0c256b76904722420f 100644
--- a/talk/app/webrtc/peerconnectioninterface.h
+++ b/talk/app/webrtc/peerconnectioninterface.h
@@ -81,6 +81,7 @@
 #include "talk/app/webrtc/umametrics.h"
 #include "webrtc/base/fileutils.h"
 #include "webrtc/base/network.h"
+#include "webrtc/base/rtccertificate.h"
 #include "webrtc/base/sslstreamadapter.h"
 #include "webrtc/base/socketaddress.h"
 
@@ -242,6 +243,7 @@ class PeerConnectionInterface : public rtc::RefCountInterface {
     TcpCandidatePolicy tcp_candidate_policy;
     int audio_jitter_buffer_max_packets;
     bool audio_jitter_buffer_fast_accelerate;
+    std::vector<rtc::scoped_refptr<rtc::RTCCertificate>> certificates;

[tommi, 2015/08/21 14:24:39]

This "struct" is outgrowing it's "structness".  Can you add a TODO for you, me
or whomever to change it to be a class and make the member variables private? 
Since we're now adding a vector, pointers to refcounted objects etc, it is
becoming more and more important to check correct usage of this type. (thread
checks etc)

[hbos, 2015/08/24 10:16:43]

Done.

 
     RTCConfiguration()
         : type(kAll),