-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Attach simulcast offer sendEncodings to suitable (addTrack) transceiver. #2155
Conversation
I like where this is going. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM % comments
Updated. Thanks for the feedback! |
<var>transceiver</var>.<a>[[\Sender]]</a>.<a>[[\sendEncodings]]</a> | ||
to <var>sendEncodings</var>, and set | ||
<var>transceiver</var>.<a>[[\Sender]]</a>.<a>[[\LastReturnedParameters]]</a> | ||
to <code>null</code>.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this mean we allow modifying sendEncodings in subsequent offers for transceivers that already exist?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, suitable = unused. See #2147 (comment).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack!
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <hta@chromium.org> Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com> Cr-Commit-Position: refs/heads/master@{#742134}
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <hta@chromium.org> Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com> Cr-Commit-Position: refs/heads/master@{#742134}
adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <hta@chromium.org> Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com> Cr-Commit-Position: refs/heads/master@{#742134}
…ceiver in simulcast, a=testonly Automatic update from web-platform-tests webrtc wpt: add test for reusing a transceiver in simulcast adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <hta@chromium.org> Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com> Cr-Commit-Position: refs/heads/master@{#742134} -- wpt-commits: 7f0a51b8417a3008d998403d3a3360ef9a4ca621 wpt-pr: 21841
…ceiver in simulcast, a=testonly Automatic update from web-platform-tests webrtc wpt: add test for reusing a transceiver in simulcast adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <hta@chromium.org> Commit-Queue: Philipp Hancke <philipp.hancke@googlemail.com> Cr-Commit-Position: refs/heads/master@{#742134} -- wpt-commits: 7f0a51b8417a3008d998403d3a3360ef9a4ca621 wpt-pr: 21841
…ceiver in simulcast, a=testonly Automatic update from web-platform-tests webrtc wpt: add test for reusing a transceiver in simulcast adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <htachromium.org> Commit-Queue: Philipp Hancke <philipp.hanckegooglemail.com> Cr-Commit-Position: refs/heads/master{#742134} -- wpt-commits: 7f0a51b8417a3008d998403d3a3360ef9a4ca621 wpt-pr: 21841 UltraBlame original commit: ee113878703ea43bfce676e738d40b2f8f655ff9
…ceiver in simulcast, a=testonly Automatic update from web-platform-tests webrtc wpt: add test for reusing a transceiver in simulcast adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <htachromium.org> Commit-Queue: Philipp Hancke <philipp.hanckegooglemail.com> Cr-Commit-Position: refs/heads/master{#742134} -- wpt-commits: 7f0a51b8417a3008d998403d3a3360ef9a4ca621 wpt-pr: 21841 UltraBlame original commit: ee113878703ea43bfce676e738d40b2f8f655ff9
…ceiver in simulcast, a=testonly Automatic update from web-platform-tests webrtc wpt: add test for reusing a transceiver in simulcast adds a test case for w3c/webrtc-pc#2155 BUG=944821 Change-Id: Idee2e83cd6dbd217802698db46fe94b6ca0f0791 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2059972 Reviewed-by: Harald Alvestrand <htachromium.org> Commit-Queue: Philipp Hancke <philipp.hanckegooglemail.com> Cr-Commit-Position: refs/heads/master{#742134} -- wpt-commits: 7f0a51b8417a3008d998403d3a3360ef9a4ca621 wpt-pr: 21841 UltraBlame original commit: ee113878703ea43bfce676e738d40b2f8f655ff9
Fixes #2147.
Preview | Diff