Add method to notify blink when PeerConnection is closed in Chrome

Source/modules/mediastream/RTCPeerConnection.cpp

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 588 matching lines @@
     if (exceptionState.hadException())
         return nullptr;
     return dtmfSender;
 }
 
 void RTCPeerConnection::close(ExceptionState& exceptionState)
 {
     if (throwExceptionIfSignalingStateClosed(m_signalingState, exceptionState))
         return;
 
-    m_peerHandler->stop();
-    m_closed = true;
-
-    changeIceConnectionState(ICEConnectionStateClosed);
-    changeIceGatheringState(ICEGatheringStateComplete);
-    changeSignalingState(SignalingStateClosed);
+    doClose();
 }
 
 void RTCPeerConnection::negotiationNeeded()
 {
     ASSERT(!m_closed);
     scheduleDispatchEvent(Event::create(EventTypeNames::negotiationneeded));
 }
 
 void RTCPeerConnection::didGenerateICECandidate(const blink::WebRTCICECandidate& webCandidate)
 {
@@ skipping 75 matching lines @@
     m_dataChannels.append(channel);
 
     scheduleDispatchEvent(RTCDataChannelEvent::create(EventTypeNames::datachannel, false, false, channel));
 }
 
 void RTCPeerConnection::releasePeerConnectionHandler()
 {
     stop();
 }
 
+void RTCPeerConnection::closePeerConnection()

[jiayl, 2014/08/15 16:12:38]

We should make sure the PC is not already closed either here or in the Chromium
caller. Up to you.

[vrk (LEFT CHROMIUM), 2014/08/16 00:36:25]

Good point. I added a few asserts here. Done.

+{
+    doClose();
+}
+
 const AtomicString& RTCPeerConnection::interfaceName() const
 {
     return EventTargetNames::RTCPeerConnection;
 }
 
 ExecutionContext* RTCPeerConnection::executionContext() const
 {
     return ActiveDOMObject::executionContext();
 }
 
@@ skipping 40 matching lines @@
 }
 
 void RTCPeerConnection::changeIceConnectionState(ICEConnectionState iceConnectionState)
 {
     if (m_iceConnectionState != ICEConnectionStateClosed && m_iceConnectionState != iceConnectionState) {
         m_iceConnectionState = iceConnectionState;
         scheduleDispatchEvent(Event::create(EventTypeNames::iceconnectionstatechange));
     }
 }
 
+void RTCPeerConnection::doClose()
+{
+    m_peerHandler->stop();
+    m_closed = true;
+
+    changeIceConnectionState(ICEConnectionStateClosed);
+    changeIceGatheringState(ICEGatheringStateComplete);
+    changeSignalingState(SignalingStateClosed);
+}
+
 void RTCPeerConnection::scheduleDispatchEvent(PassRefPtrWillBeRawPtr<Event> event)
 {
     m_scheduledEvents.append(event);
 
     m_dispatchScheduledEventRunner.runAsync();
 }
 
 void RTCPeerConnection::dispatchScheduledEvent()
 {
     if (m_stopped)
@@ skipping 14 matching lines @@
     visitor->trace(m_localStreams);
     visitor->trace(m_remoteStreams);
     visitor->trace(m_dataChannels);
 #if ENABLE(OILPAN)
     visitor->trace(m_scheduledEvents);
 #endif
     EventTargetWithInlineData::trace(visitor);
 }
 
 } // namespace blink