Removing "crypto_required" from MediaContentDescription.

webrtc/api/webrtcsession.h

Index: webrtc/api/webrtcsession.h
diff --git a/webrtc/api/webrtcsession.h b/webrtc/api/webrtcsession.h
index 94f146ff57c7e4221cda105aa501514513f66965..e011e51489f18eaa74bd3d674cab0d6557aee676 100644
--- a/webrtc/api/webrtcsession.h
+++ b/webrtc/api/webrtcsession.h
@@ -197,9 +197,11 @@ class WebRtcSession :
 
   cricket::BaseChannel* GetChannel(const std::string& content_name);
 
-  void SetSdesPolicy(cricket::SecurePolicy secure_policy);
   cricket::SecurePolicy SdesPolicy() const;
 
+  // Returns true if either DTLS or SDES is required by this session.
+  bool SecureRequired() const;

[pthatcher1, 2016/11/30 19:16:17]

"SecureRequired" sounds funny, and the logic/meaning down below is really "SRTP
is required".  So why not just call this "srtp_required" as a variable and
"srtp_required()" as a method name?

[Taylor Brandstetter, 2016/12/01 02:41:34]

This is just the name we're using elsewhere, so I used it here for consistency.
I'll change it though, I agree with you.

And for your second question: I didn't add a new variable because this can be
determined from existing variables.

+
   // Get current ssl role from transport.
   bool GetSslRole(const std::string& transport_name, rtc::SSLRole* role);