VIDEO-9282 - Migrate SDK from using node.js util.

[PikaJoyce]

Contributing to Twilio

All third party contributors acknowledge that any contributions they provide will be made under the same open source license that the open source project is provided under.

• I acknowledge that all my contributions will be made under the project's license.

Pull Request Details

In this PR, we are redefining the way we grab the global context.
global.window to window

as well as

migrating the util.inherits method into the SDK

Description

This PR partially addresses VIDEO-9282.

We have migrated the SDK dependencies off of the node.js util module as well as fixing the issue mentioned on this issue.

However, there is a larger effort and a ticket in place to address our dependencies that rely on util still which will be addressed in the future.

Burndown

Before review

• Updated CHANGELOG.md if necessary
• Added unit tests if necessary
• Updated affected documentation
• Verified locally with npm test
• Manually sanity tested running locally
• Included screenshot as PR comment (if needed)
• Ready for review

[charliesantos]

Can you please add a changelog?

[makarandp0]

@PikaJoyce, I took a quick look at new Backoff implementation and it does not look like it implements the same contract. specifically there is no way to cancel the timer started by backoff.

I have not looked at inherit implementation, but I think it would best if you can add unit tests for both backoff and inherits before replacing them, so that you can see them passing before and after your changes. that would help you figure out if new backoff and inherit implementation are doing what they are supposed to do.

[makarandp0]

@PikaJoyce - This is looking good. I have just couple of small change requests. Also please make sure that tests pass after the changes.

[makarandp0]

please add a changelog entry

[makarandp0]

need couple more changes.

[makarandp0]

This should be marked unstable only for firefox, make sure isFireFox is defined

Suggested change

	describe('called with EncodingParameters (@unstable: VIDEO-9969)', () => {
	const minAudioBitrate = 6000;
	describe(`called with EncodingParameters ${isFirefox ? ' - @unstable: VIDEO-9969' : ''}`, () => {
	const minAudioBitrate = 6000;

[makarandp0]

Thanks @PikaJoyce - Please cut an RC!