Summary: | RTCPeerConnection objects must have the same lifetime as Window | ||
---|---|---|---|
Product: | WebKit | Reporter: | Eric Carlson <eric.carlson> |
Component: | WebCore Misc. | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED WONTFIX | ||
Severity: | Normal | CC: | ggaren, nick.diego, oliver, sam |
Priority: | P2 | Keywords: | BlinkMergeCandidate |
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Bug Depends on: | |||
Bug Blocks: | 121101 |
Description
Eric Carlson
2013-09-10 11:09:44 PDT
That's a bit mysterious. What is the observable symptom this bug fixes? (In reply to comment #1) > That's a bit mysterious. What is the observable symptom this bug fixes? I asked Adam Bergkvist, one of the spec authors, about that section of the spec and he said: The RTCPeerConnection Garbage Collection section doesn't make sense to me either. I don't think that particular section has been touched since the w3c forked the WHATWG WebRTC spec. My initial thinking is that the RTCPeerConnection object should be protected when certain listeners are attached; similar to RTCDataChannel [1] (which mimics WebSocket). So I am marking this as WONTFIX and we can revisit the issue when the spec is updated. (In reply to comment #2) > (In reply to comment #1) > > That's a bit mysterious. What is the observable symptom this bug fixes? > > I asked Adam Bergkvist, one of the spec authors, about that section of the spec and he said: > > The RTCPeerConnection Garbage Collection section doesn't make sense to me either. > I don't think that particular section has been touched since the w3c forked the > WHATWG WebRTC spec. > > My initial thinking is that the RTCPeerConnection object should be protected when > certain listeners are attached; similar to RTCDataChannel [1] (which mimics WebSocket). > > So I am marking this as WONTFIX and we can revisit the issue when the spec is updated. Just to throw a wrench in the mix, it looks like the blink patch doesn't actually do what it states, it looks like what it does is change things so RTCPeerConnections won't get GC'd while the connection is not stopped. We may want that behavior, but I am not sure. |