Add PeerConnection IsClosed check.

webrtc/api/webrtcsession.cc

Index: webrtc/api/webrtcsession.cc
diff --git a/webrtc/api/webrtcsession.cc b/webrtc/api/webrtcsession.cc
index 06993085547809aa65b0847c8105bda89fc6bc3f..9f002413dbdf343daa3f656a70d66ff4c1ea2f4a 100644
--- a/webrtc/api/webrtcsession.cc
+++ b/webrtc/api/webrtcsession.cc
@@ -1475,6 +1475,22 @@ void WebRtcSession::SetIceConnectionState(
   }
 }
 
+void WebRtcSession::SetTransportController(
+    cricket::TransportController* transport_controller) {

[Taylor Brandstetter, 2016/06/21 23:50:02]

Could we just pass the TransportController into the WebRtcSession constructor?
It seems odd that the WebRtcSession constructs its own TransportController, then
it's immediately replaced.

I know this means updating webrtcsession_unittest.cc, but hopefully it's not too
bad...

[Zhi Huang, 2016/06/22 22:01:09]

This is a good point. I can do this.

+  transport_controller_.reset(transport_controller);
+  transport_controller_->SetIceRole(cricket::ICEROLE_CONTROLLED);
+  transport_controller_->SignalConnectionState.connect(
+      this, &WebRtcSession::OnTransportControllerConnectionState);
+  transport_controller_->SignalReceiving.connect(
+      this, &WebRtcSession::OnTransportControllerReceiving);
+  transport_controller_->SignalGatheringState.connect(
+      this, &WebRtcSession::OnTransportControllerGatheringState);
+  transport_controller_->SignalCandidatesGathered.connect(
+      this, &WebRtcSession::OnTransportControllerCandidatesGathered);
+  transport_controller_->SignalCandidatesRemoved.connect(
+      this, &WebRtcSession::OnTransportControllerCandidatesRemoved);
+}
+
 void WebRtcSession::OnTransportControllerConnectionState(
     cricket::IceConnectionState state) {
   switch (state) {