RESOLVED FIXED 158190
WebRTC: Imlement MediaEndpointPeerConnection::setLocalDescription() 
https://bugs.webkit.org/show_bug.cgi?id=158190
Summary WebRTC: Imlement MediaEndpointPeerConnection::setLocalDescription()
Adam Bergkvist
Reported 2016-05-29 01:45:58 PDT
RTCPeerConnection::setLocalDescription() [1] sets a local description created with createOffer() or createAnswer(). [1] https://w3c.github.io/webrtc-pc/archives/20160513/webrtc.html#dom-rtcpeerconnection-setlocaldescription
Attachments
Proposed patch (37.51 KB, patch)
2016-06-07 11:12 PDT, Adam Bergkvist 		eric.carlson: review+
DetailsFormatted DiffDiff
Patch for landing (37.63 KB, patch)
2016-06-07 23:26 PDT, Adam Bergkvist 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adam Bergkvist
Comment 1 2016-06-07 11:12:41 PDT
Created attachment 280724 [details] Proposed patch
Eric Carlson
Comment 2 2016-06-07 14:26:14 PDT
Comment on attachment 280724 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=280724&action=review > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:161 > + RefPtr<MediaEndpointSessionConfiguration> configurationSnapshot = internalLocalDescription() ? > + internalLocalDescription()->configuration()->clone() : MediaEndpointSessionConfiguration::create(); Nit: it may be worth putting internalLocalDescription() into a local instead of calling it twice. > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:258 > + unsigned previousNumberOfMediaDescriptions = internalLocalDescription() ? > + internalLocalDescription()->configuration()->mediaDescriptions().size() : 0; Ditto.
Adam Bergkvist
Comment 3 2016-06-07 23:26:03 PDT
Created attachment 280775 [details] Patch for landing
Adam Bergkvist
Comment 4 2016-06-07 23:27:00 PDT
(In reply to comment #2) > Comment on attachment 280724 [details] > Proposed patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=280724&action=review > > > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:161 > > + RefPtr<MediaEndpointSessionConfiguration> configurationSnapshot = internalLocalDescription() ? > > + internalLocalDescription()->configuration()->clone() : MediaEndpointSessionConfiguration::create(); > > Nit: it may be worth putting internalLocalDescription() into a local instead > of calling it twice. Fixed. > > Source/WebCore/Modules/mediastream/MediaEndpointPeerConnection.cpp:258 > > + unsigned previousNumberOfMediaDescriptions = internalLocalDescription() ? > > + internalLocalDescription()->configuration()->mediaDescriptions().size() : 0; > > Ditto. Fixed.
WebKit Commit Bot
Comment 5 2016-06-08 00:35:41 PDT
Comment on attachment 280775 [details] Patch for landing Clearing flags on attachment: 280775 Committed r201798: <http://trac.webkit.org/changeset/201798>
Note You need to log in before you can comment on or make changes to this bug.