Negotiate the same SRTP crypto suites for every DTLS association formed.

webrtc/p2p/base/faketransportcontroller.h

Index: webrtc/p2p/base/faketransportcontroller.h
diff --git a/webrtc/p2p/base/faketransportcontroller.h b/webrtc/p2p/base/faketransportcontroller.h
index 35af390d8124b5e7a389ae9832c1da18badaf776..9a696bca5ab1d5513b73b9a5dfef1aeb8e3307aa 100644
--- a/webrtc/p2p/base/faketransportcontroller.h
+++ b/webrtc/p2p/base/faketransportcontroller.h
@@ -35,26 +35,39 @@ class FakeTransportController : public TransportController {
   FakeTransportController()
       : TransportController(rtc::Thread::Current(),
                             rtc::Thread::Current(),
-                            nullptr) {}
+                            nullptr,
+                            /*redetermine_role_on_ice_restart=*/true,

[Taylor Brandstetter, 2017/04/17 22:00:34]

I know we don't use this style typically, but I think we should allow it (when
sensible) since it's more consistent with Google style and it's enforceable by
clang-tidy
(https://clang.llvm.org/extra/clang-tidy/checks/misc-argument-comment.html).
WDYT?

[pthatcher1, 2017/04/17 22:12:41]

I don't like the look of it, but it seems like a good choice in this instance
since it's difficult to make a named variable to pass into  the constructor.

+                            rtc::CryptoOptions()) {}
 
   explicit FakeTransportController(bool redetermine_role_on_ice_restart)
       : TransportController(rtc::Thread::Current(),
                             rtc::Thread::Current(),
                             nullptr,
-                            redetermine_role_on_ice_restart) {}
+                            redetermine_role_on_ice_restart,
+                            rtc::CryptoOptions()) {}
 
   explicit FakeTransportController(IceRole role)
       : TransportController(rtc::Thread::Current(),
                             rtc::Thread::Current(),
-                            nullptr) {
+                            nullptr,
+                            /*redetermine_role_on_ice_restart=*/true,
+                            rtc::CryptoOptions()) {
     SetIceRole(role);
   }
 
   explicit FakeTransportController(rtc::Thread* network_thread)
-      : TransportController(rtc::Thread::Current(), network_thread, nullptr) {}
+      : TransportController(rtc::Thread::Current(),
+                            network_thread,
+                            nullptr,
+                            /*redetermine_role_on_ice_restart=*/true,
+                            rtc::CryptoOptions()) {}
 
   FakeTransportController(rtc::Thread* network_thread, IceRole role)
-      : TransportController(rtc::Thread::Current(), network_thread, nullptr) {
+      : TransportController(rtc::Thread::Current(),
+                            network_thread,
+                            nullptr,
+                            /*redetermine_role_on_ice_restart=*/true,
+                            rtc::CryptoOptions()) {
     SetIceRole(role);
   }