Removing "crypto_required" from MediaContentDescription.

webrtc/api/webrtcsession.h

 /*
  *  Copyright 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 179 matching lines @@
   }
   virtual cricket::VideoChannel* video_channel() {
     return video_channel_.get();
   }
   virtual cricket::DataChannel* data_channel() {
     return data_channel_.get();
   }
 
   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);
 
   // Get current SSL role for this channel's transport.
   // If |transport| is null, returns false.
   bool GetSslRole(const cricket::BaseChannel* channel, rtc::SSLRole* role);
 
   void CreateOffer(
       CreateSessionDescriptionObserver* observer,
       const PeerConnectionInterface::RTCOfferAnswerOptions& options,
@@ skipping 307 matching lines @@
 
 #ifdef HAVE_QUIC
   std::unique_ptr<QuicDataTransport> quic_data_transport_;
 #endif  // HAVE_QUIC
 
   RTC_DISALLOW_COPY_AND_ASSIGN(WebRtcSession);
 };
 }  // namespace webrtc
 
 #endif  // WEBRTC_API_WEBRTCSESSION_H_