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

Get media fails if Bubble Notification enabled in Chrome Beta 37 #15

Open
warrenjmcdonald opened this issue Aug 5, 2014 · 7 comments
Open

Comments

@warrenjmcdonald
Copy link

I have done a comparison using non SSL and a regular call to getusermedia works but if media is initialised through rtc-media the page reports unable get media. No error is raised in the JS console

Turning notification back to default allows media to be captured again.

@DamonOehlman
Copy link
Member

@warrenjmcdonald Is this in conjunction with the rtc-captureconfig module? I made some changes to package yesterday that to move generated constraints to the optional section (rather than mandatory) which was causing failures - @silviapfeiffer picked this up in one of our apps...

@warrenjmcdonald
Copy link
Author

It is using captureconfig which is/was also causing grief on mobiles if we
set min res too high. So optional would be better.

Not sure if this is connected with the bubble notification issue unless
they have changed underlying rules.

We can get this module updated to test when doing next bundle.
On 05/08/2014 8:40 PM, "Damon Oehlman" notifications@github.com wrote:

@warrenjmcdonald https://github.com/warrenjmcdonald Is this in
conjunction with the rtc-captureconfig
https://github.com/rtc-io/rtc-captureconfig module? I made some changes
to package yesterday that to move generated constraints to the optional
section (rather than mandatory) which was causing failures -
@silviapfeiffer https://github.com/silviapfeiffer picked this up in one
of our apps...


Reply to this email directly or view it on GitHub
#15 (comment).

@silviapfeiffer
Copy link
Member

What is this bubble notification thing you're talking about @warrenjmcdonald ? Is that the new getUserMedia access request bubble as in chrome://flags/#enable-permissions-bubbles ? If so, that may well be a chrome bug. Did you test on apprtc, too?

@silviapfeiffer
Copy link
Member

Incidentally, I just tried turning the bubbles on and I got an error when first loading it that way. It worked on reload though.

@warrenjmcdonald
Copy link
Author

@silviapfeiffer yes it is the new GUM notification. I have tried with other code and do not get the error. We only getting the issue when using the rtc-media capture ,method. Other libs like SIP.js and older VCC pages using getUserMedia npm package ( https://www.npmjs.org/package/getusermedia ) don't have the problem,
It could be browser bug or method compatibility issue. Not a blocker, just really a heads up to report potential problem if this is turned on for testing

@silviapfeiffer
Copy link
Member

@warrenjmcdonald that is indeed odd. Did it persist or go away after a reload, like it did for me?

@DamonOehlman
Copy link
Member

I'll definitely have a look at this at some stage, thanks for reporting it Warren!

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

3 participants