Bug 191188

Summary: Add support for RTCMuxPolicy
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebRTCAssignee: youenn fablet <youennf>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric.carlson, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

youenn fablet
Reported 2018-11-02 04:56:53 PDT
Add support for RTCMuxPolicy
Attachments
Patch (20.00 KB, patch)
2018-11-02 05:00 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Patch (20.83 KB, patch)
2018-11-04 00:17 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2018-11-02 05:00:06 PDT
Created attachment 353688 [details] Patch
youenn fablet
Comment 2 2018-11-02 07:01:18 PDT
Comment on attachment 353688 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=353688&action=review > Source/WebCore/platform/mediastream/MediaEndpointConfiguration.cpp:39 > +MediaEndpointConfiguration::MediaEndpointConfiguration(Vector<IceServerInfo>&& iceServers, RTCIceTransportPolicy iceTransportPolicy, RTCBundlePolicy bundlePolicy, RTCPMuxPolicy rtcpMuxPolicy, unsigned short iceCandidatePoolSize, Vector<CertificatePEM>&& certificates) We might be able to remove this constructor. I'll try that in a separate patch. > Source/WebCore/platform/mediastream/RTCPMuxPolicy.h:2 > + * Copyright (C) 2017 Apple Inc. All rights reserved. Will fix it to 2018.
youenn fablet
Comment 3 2018-11-04 00:17:11 PDT
Created attachment 353799 [details] Patch
WebKit Commit Bot
Comment 4 2018-11-04 00:55:09 PDT
Comment on attachment 353799 [details] Patch Clearing flags on attachment: 353799 Committed r237778: <https://trac.webkit.org/changeset/237778>
WebKit Commit Bot
Comment 5 2018-11-04 00:55:11 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2018-11-04 00:56:22 PDT
<rdar://problem/45789577>
Note You need to log in before you can comment on or make changes to this bug.