Adding support for Unified Plan offer/answer negotiation.

webrtc/p2p/base/sessiondescription.h

 /*
  *  Copyright 2004 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 38 matching lines @@
               ContentDescription* description)
       : name(name),
         type(type),
         rejected(rejected),
         bundle_only(bundle_only),
         description(description) {}
   std::string name;
   std::string type;
   bool rejected = false;
   bool bundle_only = false;
+  size_t msection_index = -1;

[Taylor Brandstetter, 2017/08/09 17:05:11]

It's redundant to store this inside ContentInfo, since it's implied by the
ContentInfo's index in SessionDescription::contents().

Where this is used in IsDtlsActive, you could do this instead:

int msection_index = content - &current_description->contents()[0];

Since subtraction of pointers gives you the number of elements that fit between
the two addresses.

Or you could just pass the index into the method.

[Zhi Huang, 2017/08/09 21:51:02]

This is a good point!

   ContentDescription* description = nullptr;
 };
 
 typedef std::vector<std::string> ContentNames;
 
 // This class provides a mechanism to aggregate different media contents into a
 // group. This group can also be shared with the peers in a pre-defined format.
 // GroupInfo should be populated only with the |content_name| of the
 // MediaDescription.
 class ContentGroup {
@@ skipping 128 matching lines @@
 
 // Indicates whether a ContentDescription was sent by the local client
 // or received from the remote client.
 enum ContentSource {
   CS_LOCAL, CS_REMOTE
 };
 
 }  // namespace cricket
 
 #endif  // WEBRTC_P2P_BASE_SESSIONDESCRIPTION_H_