Description of changes: With previous changes we would hit this.context.currentVideoSendCodec === undefined if attendee capability is Send (yes, JS Strict mode would have caught this :) ).
Testing:
Can send using Send AC again.
Can these tested using a demo application? Please provide reproducible step-by-step instructions.
Pick Send for Content AC in additional options
Send content share and verify on remote attendee
Checklist:
Have you successfully run npm run build:release locally?
y
Do you add, modify, or delete public API definitions? If yes, has that been reviewed and approved?
n
Do you change the wire protocol, e.g. the request method? If yes, has that been reviewed and approved?
n
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Issue #: None
Description of changes: With previous changes we would hit
this.context.currentVideoSendCodec === undefined
if attendee capability is Send (yes, JS Strict mode would have caught this :) ).Testing:
Can send using Send AC again.
Can these tested using a demo application? Please provide reproducible step-by-step instructions.
Checklist:
Have you successfully run
npm run build:release
locally? yDo you add, modify, or delete public API definitions? If yes, has that been reviewed and approved? n
Do you change the wire protocol, e.g. the request method? If yes, has that been reviewed and approved? n
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.