Allow PeerConnection to be garbage collected after close().

LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html

Index: LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html
diff --git a/LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html b/LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html
index 076e0d3e553f4cacd149eb520ef97d93f59698a3..9e9ac3f37465ab69f8ecb7a963010268c07a8fea 100644
--- a/LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html
+++ b/LayoutTests/fast/mediastream/RTCPeerConnection-lifetime.html
@@ -24,7 +24,7 @@ function dataChannelOpen()
     shouldNotThrow("dc.send('xyzzy');");
 }
 
-function createPeerConnection()
+function createPeerConnectionAndDataChannel()
 {
     // The Peer Connection object is leaked
     var pc = new webkitRTCPeerConnection({iceServers:[]}, null);
@@ -32,7 +32,24 @@ function createPeerConnection()
     dc.onopen = dataChannelOpen;
 }
 
-createPeerConnection();
+// Test that the PeerConnection object is gc'd if close is called.
+var pcA = new webkitRTCPeerConnection(null, null);
+observationA = internals.observeGC(pcA);
+pcA.close();
+pcA = null;
+gc();

[keishi, 2014/06/17 08:16:42]

I think its better to use asyncGC(). It ensures that precise garbage collection
is run (instead of the conservative stack scanning).

[Henrik Grunell, 2014/06/17 09:08:19]

Done.

+shouldBeTrue('observationA.wasCollected');
+observationA = null;
+
+// Test that the PeerConnection object is not gc'd if close is not called.
+var pcB = new webkitRTCPeerConnection(null, null);
+observationB = internals.observeGC(pcB);
+pcB = null;
+gc();
+shouldBeFalse('observationB.wasCollected');
+observationB = null;
+
+createPeerConnectionAndDataChannel();
 gc();
 
 window.jsTestIsAsync = true;