Bug 167571

Summary: [WebRTC] RTCPeerConnection should reject getStats promise in closed state
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebRTCAssignee: youenn fablet <youennf>
Status: RESOLVED INVALID    
Severity: Normal CC: adam.bergkvist, agouaillard, jonlee, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 143211    
Attachments:
Description Flags
Patch achristensen: review+

youenn fablet
Reported 2017-01-29 21:32:55 PST
RTCPeerConnection should reject getStats promise in closed mode
Attachments
Patch (3.95 KB, patch)
2017-01-29 21:34 PST, youenn fablet 		achristensen: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2017-01-29 21:34:37 PST
Created attachment 300085 [details] Patch
Alex Christensen
Comment 2 2017-01-30 15:37:34 PST
Comment on attachment 300085 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=300085&action=review > Source/WebCore/Modules/mediastream/RTCPeerConnection.cpp:388 > + promise.reject(Exception { INVALID_STATE_ERR, "Cannot get stats on closed state" }); Do we need a message here? "Cannot get stats on closed state" doesn't make much sense.
Adam Bergkvist
Comment 3 2017-02-09 04:47:13 PST
The getStats() method [1] doesn't specify that it should reject on closed state. I believe it's a feature that you should be able to extract the "last" stats gathered by a closed RTCPeerConnection. [1] https://w3c.github.io/webrtc-pc/archives/20161219/webrtc.html#rtcpeerconnection-interface-extensions-2
Radar WebKit Bug Importer
Comment 4 2017-08-29 09:51:51 PDT
<rdar://problem/34134439>
Jon Lee
Comment 5 2017-08-29 15:43:59 PDT
The spec has changed since.
Note You need to log in before you can comment on or make changes to this bug.