Accept SDP with TRANSPORT attributes missing from bundled m= sections.

webrtc/p2p/base/transportdescriptionfactory.h

Index: webrtc/p2p/base/transportdescriptionfactory.h
diff --git a/webrtc/p2p/base/transportdescriptionfactory.h b/webrtc/p2p/base/transportdescriptionfactory.h
index 2dde5b12584b729f6a59963dedd76a82058fc942..8a530decb4b89435a236d418d45aa2316162b8f0 100644
--- a/webrtc/p2p/base/transportdescriptionfactory.h
+++ b/webrtc/p2p/base/transportdescriptionfactory.h
@@ -53,10 +53,15 @@ class TransportDescriptionFactory {
   TransportDescription* CreateOffer(const TransportOptions& options,
       const TransportDescription* current_description) const;
   // Create a transport description that is a response to an offer.
+  // |bundled| indicates that this transport description is being generated
+  // for an m= section that's being bundled into another m= section, in

[pthatcher1, 2017/02/17 19:17:32]

might be more clear as 

"|bundled| indicates that this transport description is in an m= section that's
in a bundle group and isn't the first m= section in the bundle group, so it
doesn't need transport-level attributes".

Better yet, why not just call the variable |include_transport_level_attributes|
or some shorter version of that?  Then you don't need the comment.

[Taylor Brandstetter, 2017/02/17 21:49:14]

Done.

+  // which case transport-level attributes aren't needed since they can
+  // be found in the other m= section.
   TransportDescription* CreateAnswer(
       const TransportDescription* offer,
       const TransportOptions& options,
-      const TransportDescription* current_description) const;
+      const TransportDescription* current_description,
+      bool bundled) const;

[pthatcher1, 2017/02/17 19:17:32]

It ought to be the 3rd argument, not the last.  And maybe it should be part of
TransportOptions.

[Taylor Brandstetter, 2017/02/17 21:49:14]

Done (though not the TransportOptions part).

 
  private:
   bool SetSecurityInfo(TransportDescription* description,