RESOLVED FIXED 189270
Make LibWebRTCRtpSenderBackend own its libwebrtc RTP sender backend 
https://bugs.webkit.org/show_bug.cgi?id=189270
Summary Make LibWebRTCRtpSenderBackend own its libwebrtc RTP sender backend
youenn fablet
Reported Tuesday, September 4, 2018 9:01:37 PM UTC
Make LibWebRTCRtpSenderBackend own its libwebrtc RTP sender backend
Attachments
Patch (21.29 KB, patch)
2018-09-04 15:31 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch for landing (21.32 KB, patch)
2018-09-04 16:31 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 Tuesday, September 4, 2018 11:31:10 PM UTC
Created attachment 348857 [details] Patch
Eric Carlson
Comment 2 Wednesday, September 5, 2018 12:21:00 AM UTC
Comment on attachment 348857 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=348857&action=review > Source/WebCore/ChangeLog:8 > + Previously, the libwebrtc sender were owned in LibWebRTCMediaEndpoint. Nit: sender -> senders > Source/WebCore/ChangeLog:9 > + This patch makes them owned by LibWebRTCRtpSenderBackend. Nit: makes them owned by -> makes them be owned by > Source/WebCore/ChangeLog:11 > + This will help simplifying the implementation of RTCRtpSender::replaceTrack. Nit: simplifying -> simplify > Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:258 > + if (m_peerConnectionBackend.shouldOfferAllowToReceive("audio")) Nit: "audio" -> "audio"_s > Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:260 > + if (m_peerConnectionBackend.shouldOfferAllowToReceive("video")) Nit: "video" -> "video"_s
youenn fablet
Comment 3 Wednesday, September 5, 2018 12:31:51 AM UTC
Created attachment 348870 [details] Patch for landing
WebKit Commit Bot
Comment 4 Wednesday, September 5, 2018 1:04:07 AM UTC
Comment on attachment 348870 [details] Patch for landing Clearing flags on attachment: 348870 Committed r235649: <https://trac.webkit.org/changeset/235649>
WebKit Commit Bot
Comment 5 Wednesday, September 5, 2018 1:04:09 AM UTC
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 Wednesday, September 5, 2018 1:06:16 AM UTC
<rdar://problem/44118910>
Note You need to log in before you can comment on or make changes to this bug.