RESOLVED FIXED 158691
WebRTC: Add media setup test where media is set up in one direction at a time 
https://bugs.webkit.org/show_bug.cgi?id=158691
Summary WebRTC: Add media setup test where media is set up in one direction at a time
Adam Bergkvist
Reported Monday, June 13, 2016 1:26:35 PM UTC
Setting up media in one direction at a time means completing a first offer/answer dialog with unidirectional media, and then sending an updated offer to add media in the other direction.
Attachments
Proposed patch (16.01 KB, patch)
2016-06-14 02:29 PDT, Adam Bergkvist 		eric.carlson: review+
DetailsFormatted DiffDiff
Patch for landing (16.00 KB, patch)
2016-06-14 06:44 PDT, Adam Bergkvist 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Bergkvist
Comment 1 Tuesday, June 14, 2016 10:29:33 AM UTC
Created attachment 281248 [details] Proposed patch
Eric Carlson
Comment 2 Tuesday, June 14, 2016 1:59:08 PM UTC
Comment on attachment 281248 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=281248&action=review > LayoutTests/fast/mediastream/RTCPeerConnection-media-setup-two-dialogs.html:12 > + description("Test setting up media between two RTCPeerConnection instances with one single SDP dialog."); Nit: "... with one single ..." -> "... with a single ..." > LayoutTests/fast/mediastream/RTCPeerConnection-media-setup-two-dialogs.html:112 > + testPassed("Frist offer/answer dialog completed") Nit: "Frist" -> "First"
Adam Bergkvist
Comment 3 Tuesday, June 14, 2016 2:44:18 PM UTC
Created attachment 281254 [details] Patch for landing
Adam Bergkvist
Comment 4 Tuesday, June 14, 2016 2:45:43 PM UTC
(In reply to comment #2) > Comment on attachment 281248 [details] > Proposed patch Thanks for reviewing Eric. 'Patch for landing' submitted. > View in context: > https://bugs.webkit.org/attachment.cgi?id=281248&action=review > > > LayoutTests/fast/mediastream/RTCPeerConnection-media-setup-two-dialogs.html:12 > > + description("Test setting up media between two RTCPeerConnection instances with one single SDP dialog."); > > Nit: "... with one single ..." -> "... with a single ..." Fixed. > > LayoutTests/fast/mediastream/RTCPeerConnection-media-setup-two-dialogs.html:112 > > + testPassed("Frist offer/answer dialog completed") > > Nit: "Frist" -> "First" Fixed.
WebKit Commit Bot
Comment 5 Tuesday, June 14, 2016 4:11:43 PM UTC
Comment on attachment 281254 [details] Patch for landing Clearing flags on attachment: 281254 Committed r202043: <http://trac.webkit.org/changeset/202043>
WebKit Commit Bot
Comment 6 Tuesday, June 14, 2016 4:11:47 PM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.