Title: [134413] branches/chromium/1312/Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp
Revision
134413
Author
[email protected]
Date
2012-11-13 07:15:03 -0800 (Tue, 13 Nov 2012)

Log Message

Merge 134093 - MediaStream API: Don't trigger any object deletion during RTCPeerConnection::stop
https://bugs.webkit.org/show_bug.cgi?id=101586

Reviewed by Adam Barth.

Stop calling stop on the RTCPeerConnectionHandler, and don't delete it, when
ActiveDOMObject::stop is called on RTCPeerConnection. Due to the async nature of the new
API some WebCore objects might be cleaned away which is not allowed at this stage.

This behaviour is not possible to test unfortunately in webkit.

* Modules/mediastream/RTCPeerConnection.cpp:
(WebCore::RTCPeerConnection::stop):


[email protected]
Review URL: https://codereview.chromium.org/11363215

Modified Paths

Diff

Modified: branches/chromium/1312/Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp (134412 => 134413)


--- branches/chromium/1312/Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp	2012-11-13 15:11:40 UTC (rev 134412)
+++ branches/chromium/1312/Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp	2012-11-13 15:15:03 UTC (rev 134413)
@@ -523,11 +523,6 @@
 
 void RTCPeerConnection::stop()
 {
-    if (m_readyState != ReadyStateClosed)
-        m_peerHandler->stop();
-
-    m_peerHandler.clear();
-
     m_iceState = IceStateClosed;
     m_readyState = ReadyStateClosed;
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to