-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
implement getDisplayMedia #424
base: master
Are you sure you want to change the base?
Conversation
…Controller interface to allow getDisplayMedia videoTrack.enabled support and RTCMediaStreamTrack.videoCaptureController.stopCapture support
That was easy LOL |
@hthetiot tested and working fine i merged it in a fork, But after closing the streaming the app orientation seems like locked and can't be switched anymore until you restart the app |
@wnabil thank you for testing, the current limitations is you can only capture the current application. |
@hthetiot hi there long time no talk. hope u r doing well. Any idea whn this might be released. It looks great |
Hello there hope you doing well. This is limited to sharing the current application screen not the full screen with app in the background. I can may be debase on master and if it works i can release after #687 |
@hthetiot I think that is a good start. And once these classes in place may be we will help with the ReplayKit Broadcast extension |
Hi @hthetiot |
@samgabriel @igormartsekha I merged master (M87) feel free to make PR on the same branch. |
This pull request introduces 2 alerts when merging 7697788 into c4f0217 - view on LGTM.com new alerts:
|
Via https://api.travis-ci.com/v3/job/521179554/log.txt Need to fix error due merge master. Will do asap, AFK. |
task/getDisplayMedia - 2 Bugfixes
Note that last build fail cause travis balance on ios. |
…-plugin-iosrtc into task/getDisplayMedia
Capture
navigator.mediaDevices.getDisplayMedia
test result of prototype receiving screen capture from ios to chrome:Testing
To test
task/getDisplayMedia
branch