Implementing unified plan encoding of msid.

talk/app/webrtc/webrtcsdp.h

Index: talk/app/webrtc/webrtcsdp.h
diff --git a/talk/app/webrtc/webrtcsdp.h b/talk/app/webrtc/webrtcsdp.h
index fcbbdad3d3a7e2b9913cf5a45d0f8f626aec8ac7..a2843550e9b712d3d00526a8eedd4b30778387b7 100644
--- a/talk/app/webrtc/webrtcsdp.h
+++ b/talk/app/webrtc/webrtcsdp.h
@@ -50,8 +50,10 @@ struct SdpParseError;
 // Serialize SessionDescription including candidates if
 // JsepSessionDescription has candidates.
 // jdesc - The JsepSessionDescription object to be serialized.
+// unified_plan_sdp - If set to true, include "a=msid" lines where appropriate.
 // return - SDP string serialized from the arguments.
-std::string SdpSerialize(const JsepSessionDescription& jdesc);
+std::string SdpSerialize(const JsepSessionDescription& jdesc,
+                         bool unified_plan_sdp = false);

[pthatcher1, 2016/02/12 03:54:35]

Rather than have a default variable, can we make it explicit?  

In fact, it might even be nice to just have two functions:

SdpSerliazePlanB
SdpSerliazeUnifiedPlan

[Taylor Brandstetter, 2016/02/12 20:44:57]

Done.

 
 // Serializes the passed in IceCandidateInterface to a SDP string.
 // candidate - The candidate to be serialized.