Bug 170057 – Add support for RTCRtpReceiver/RTCRtpSender getParameters

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[youenn fablet]

Add support for RTCRtpReceiver/RTCRtpSender getParameters

[youenn fablet]

Created attachment 305291 [details]
Patch

[youenn fablet]

Created attachment 305332 [details]
Patch

[Build Bot]

Attachment 305332 [details] did not pass style-queue:


ERROR: Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCMediaEndpoint.cpp:475:  Should be indented on a separate line, with the colon or comma first on that line.  [whitespace/indent] [4]
ERROR: Source/ThirdParty/libwebrtc/Source/webrtc/api/mediatypes.h:26:  The parameter name "type" adds no information, so it should be removed.  [readability/parameter_name] [5]
Total errors found: 2 in 25 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Build Bot]

Comment on attachment 305332 [details]
Patch

Attachment 305332 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/3406545

New failing tests:
http/tests/preload/download_resources.html

[Build Bot]

Created attachment 305350 [details]
Archive of layout-test-results from ews114 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-elcapitan  Platform: Mac OS X 10.11.6

[youenn fablet]

Test error is not related

[Alex Christensen]

Comment on attachment 305332 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=305332&action=review

> Source/WebCore/Modules/mediastream/RTCRtpParameters.h:50
> +        unsigned short payloadType { 0};

space after 0 in a few places in this file

[youenn fablet]

Created attachment 305480 [details]
Patch for landing

[youenn fablet]

Thanks for the review.

(In reply to Alex Christensen from comment #7)
> Comment on attachment 305332 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=305332&action=review
> 
> > Source/WebCore/Modules/mediastream/RTCRtpParameters.h:50
> > +        unsigned short payloadType { 0};
> 
> space after 0 in a few places in this file

Done

[WebKit Commit Bot]

Comment on attachment 305480 [details]
Patch for landing

Clearing flags on attachment: 305480

Committed r214420: <http://trac.webkit.org/changeset/214420>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Alexey Proskuryakov]

webrtc/video-getParameters.html fails every time on Apple Sierra Release WK2 (Tests)

-PASS Basic video stats 
+FAIL Basic video stats assert_equals: testing sender vs. receiver parameters expected "{\"codecs\":[],\"degradationPreference\":\"balanced\",\"encodings\":[{\"active\":true,\"dtx\":\"disabled\",\"fec\":{\"ssrc\":57},\"maxBitrate\":0,\"maxFramerate\":0,\"priority\":\"medium\",\"rid\":\"\",\"rtx\":{\"ssrc\":3001127280},\"scaleResolutionDownBy\":1,\"ssrc\":2378346063}],\"headerExtensions\":[],\"transactionId\":\"\"}" but got "{\"codecs\":[],\"degradationPreference\":\"balanced\",\"encodings\":[{\"active\":true,\"dtx\":\"disabled\",\"fec\":{\"ssrc\":81},\"maxBitrate\":0,\"maxFramerate\":0,\"priority\":\"medium\",\"rid\":\"\",\"rtx\":{\"ssrc\":2940112664},\"scaleResolutionDownBy\":1,\"ssrc\":2378346063}],\"headerExtensions\":[],\"transactionId\":\"\"}"