Skip to content
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

Pluginfree-Screen-Sharing.html #3

Closed
jonschull opened this issue Mar 2, 2013 · 3 comments
Closed

Pluginfree-Screen-Sharing.html #3

jonschull opened this issue Mar 2, 2013 · 3 comments

Comments

@jonschull
Copy link

  1. your online Pluginfree-Screen-Sharing.html Works! Congratulations
  2. The file of that name in your github directory is not the same file (it attempts video)
  3. Even when I use the online version of Pluginfree-Screen-Sharing.html on my own server, it doesn't work. (The button does nothing. And the private link goes back to google drive).
  4. Help?

jschull@gmail.com

@muaz-khan
Copy link
Owner

Test it on HTTPS. Because, screen capturing (currently) only works on SSL domains.

Chrome denies request automatically in the following cases:

  1. Screen capturing is not enabled via command line switch.
    mandatory: {chromeMediaSource: 'screen'} must be there
  2. Audio stream was requested (it's not supported yet).
navigator.webkitGetUserMedia({
    audio: false    /* MUST be false because audio capturer not works with screen capturer */
});
  1. Request from a page that was not loaded from a secure origin.

Here is their C++ code that denies screen capturing:

if (!screen_capture_enabled ||
    request.audio_type != content::MEDIA_NO_SERVICE ||
    !request.security_origin.SchemeIsSecure()) {
        callback.Run(content::MediaStreamDevices());
        return;
    }

Personally I don’t know why they deny non-SSL requests. Maybe they’re using iframes in sandbox mode or something else that runs only on HTTPS.

Browsers who don't understand {chromeMediaSource: 'screen'} constraint will simply get video like chrome stable or Firefox.

Can you explain what you meant by "The file of that name in your github directory is not the same file"?

@muaz-khan
Copy link
Owner

Corrected. Thanks.

@jonschull
Copy link
Author

myintthu pushed a commit to myintthu/ddd that referenced this issue Aug 28, 2015
Thanks @jonschull for this issue:
muaz-khan/WebRTC-Experiment#3

Added {chromeMediaSource: 'screen'} in the conference-ui.js file.
myintthu pushed a commit to myintthu/ddd that referenced this issue Aug 28, 2015
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

No branches or pull requests

2 participants