RTCRtpSender.setParameters() does set active parameter
authoryouenn@apple.com <youenn@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Dec 2018 02:14:31 +0000 (02:14 +0000)
committeryouenn@apple.com <youenn@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 22 Dec 2018 02:14:31 +0000 (02:14 +0000)
commit43d6d4ab47bbcf8c0e1e287ab26202148320a808
tree75a8382b0d4332db567b7359622c86b99789dc49
parent9da01364f73d064c259bdf2942023aa039249530
RTCRtpSender.setParameters() does set active parameter
https://bugs.webkit.org/show_bug.cgi?id=192848

Reviewed by Eric Carlson.

Source/WebCore:

Covered by updated test.

* Modules/mediastream/libwebrtc/LibWebRTCUtils.cpp:
(WebCore::updateRTCRtpSendParameters):
The routine was updating the local value, not the out parameter.

LayoutTests:

* webrtc/video.html:
Add a check for active value.
Test video freezing through canvas instead of stats.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239532 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/webrtc/video.html
Source/WebCore/ChangeLog
Source/WebCore/Modules/mediastream/libwebrtc/LibWebRTCUtils.cpp