Implement the "needs-ice-restart" logic for SetConfiguration.

webrtc/api/webrtcsession.h

Index: webrtc/api/webrtcsession.h
diff --git a/webrtc/api/webrtcsession.h b/webrtc/api/webrtcsession.h
index 94f146ff57c7e4221cda105aa501514513f66965..d22203035102c335dd1c32c04cc296d1fd73bd38 100644
--- a/webrtc/api/webrtcsession.h
+++ b/webrtc/api/webrtcsession.h
@@ -278,8 +278,16 @@ class WebRtcSession :
 
   cricket::DataChannelType data_channel_type() const;
 
+  // Returns true if there was an ICE restart initiated by the remote offer.
   bool IceRestartPending(const std::string& content_name) const;
 
+  // Set the "needs-ice-restart" flag as described in JSEP.
+  void SetNeedsIceRestartFlag();
+  // Returns true if the ICE restart flag above was set, and no ICE restart has
+  // occurred yet for |transport_name|. If the transport has been deleted as a
+  // result of bundling, returns false.
+  bool NeedsIceRestart(const std::string& transport_name) const;

[pthatcher1, 2016/12/10 02:26:31]

Can we just call these MIDs?  If not, why is it content_name in one place and
transport_name in the other?

[Taylor Brandstetter, 2016/12/10 19:00:35]

I'll call it content_name here too for consistency. We can change it to "mid"
later, doing it everywhere at once if we do it.

+
   // Called when an RTCCertificate is generated or retrieved by
   // WebRTCSessionDescriptionFactory. Should happen before setLocalDescription.
   void OnCertificateReady(