Skip to content

Prevent override mediaDevices on iOS 14.3#16

Merged
noahmehl merged 1 commit intomasterfrom
twilio-support
Jan 26, 2021
Merged

Prevent override mediaDevices on iOS 14.3#16
noahmehl merged 1 commit intomasterfrom
twilio-support

Conversation

@JustDoIt9
Copy link
Copy Markdown
Contributor

No description provided.

@hthetiot
Copy link
Copy Markdown
Contributor

Let me know @JustDoIt9 if you think some of your PR contains fixes that are missing on cordova-plugin-iosrtc and I will backport.

navigator.mediaDevices.getUserMedia = getUserMedia;
navigator.mediaDevices.enumerateDevices = enumerateDevices;

// Prevent WebRTC-adapter to overide navigator.mediaDevices after shim is applied since ios 14.3
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

console.log('requestPermission.result', result);
});

MediaStreamTrack.prototype.clone = function () {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have clone in iosRTC i think now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is using GoogleWebRTC from cocoapods.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, we did implement real clone lately I think and it's Twillio friendly.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@noahmehl noahmehl merged commit 93b3edc into master Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants