873854 - webrtc video not working for Logitech QuickCam S5500

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, defect, P3)

Description

[firefoxiscute]

Attachment: error-image.png

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:17.0) Gecko/20100101 Firefox/17.0 (Beta/Release)

Steps to reproduce:

Hardware:
$ lsusb
Bus 002 Device 003: ID 046d:09a1 Logitech, Inc. QuickCam Communicate MP/S5500
OS: Linux Kubuntu 13.04

1. started nightly 24.0a1 (2013-05-19)
2. opened http://mozilla.github.io/webrtc-landing/gum_test.html
3. Clicked on "Video" button (behaviour the same for Video & Audio)
4. allowed to use "UVC Camera (046d:09a1)" in small window from url bar

(btw audio worked with mic from cam)


Actual results:

no video just strange colors (changing perhaps thrice per second) could be seen


Expected results:

See video from cam I expect

Comment 1

[firefoxiscute]

Attachment: log

The log from stdout and stderr while running the page

Comment 2

[firefoxiscute]

Note that the browser I sent the bug report with is not the one I encountered the bug.

Comment 3

[Randell Jesup [:jesup] (needinfo me)]

Could you repeat the test with NSPR_LOG_MESSAGES=getusermedia:5,mediamanager:5  ?

Comment 4

[firefoxiscute]

Attachment: log with NSPR_LOG_MESSAGES=getusermedia:5,mediamanager:5

ran with mozconfig:
ac_add_options --enable-debug
export NSPR_LOG_MESSAGES=getusermedia:5,mediamanager:5
mk_add_options NSPR_LOG_MESSAGES=getusermedia:5,mediamanager:5

Comment 5

[Randell Jesup [:jesup] (needinfo me)]

Sorry - to get useful logging, you;ll need to download a Nightly Debug build.  They're available on the FTP site.

http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/

Comment 6

[firefoxiscute]

Attachment: next try :-)

I assume NSPR_LOG_MESSAGES should be an environment variable set at execution time.
Used 24.0a1 (2013-05-20)

Comment 7

[Randell Jesup [:jesup] (needinfo me)]

That's correct - NSPR_LOG_MESSAGES has limited use in non-debug builds as most debugs don't get compiled in.  You can also re-route the NSPR logs with NSPR_LOG_FILE (separate from stdout/err debugs).

Comment 8

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 751848 [details]
next try :-)

Use "-no-remote", not --no-remote

Something's wrong; this should have printed a bunch of debugs

Verify it's running the build you think (Help/About Nightly)

Ugh.... NSPR_LOG_MODULES, not MESSAGES.  Sorry.

Comment 9

[firefoxiscute]

Attachment: nspr log

This time the NSPR log.
Additional to the steps in the Bug Description I performed:
5. Click on "Stop" button

Browser version verified, still used "--no-remote".

Comment 10

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 751863 [details]
nspr log

Ok, thanks.  That shows what's going on.

This webcam is an odd beast.  It supports "unknown", MJPEG, and YUY2 video formats, while we need I420 internally (aka YCrCb/YUV 4:2:0).  YUY2 is apparently YUV 4:4:4, so the conversion is easy.

I'll file a new bug to add support for incoming video format conversion.  Thanks!

Comment 11

[Timothy B. Terriberry (:derf)]

(In reply to Randell Jesup [:jesup] from comment #10)
> formats, while we need I420 internally (aka YCrCb/YUV 4:2:0).  YUY2 is
> apparently YUV 4:4:4, so the conversion is easy.

Uh, YUY2 is 4:2:2, not 4:4:4.

Also pretty sure we use Y'CbCr internally, not CrCb.

Comment 12

[Randell Jesup [:jesup] (needinfo me)]

(In reply to Timothy B. Terriberry (:derf) from comment #11)
> (In reply to Randell Jesup [:jesup] from comment #10)
> > formats, while we need I420 internally (aka YCrCb/YUV 4:2:0).  YUY2 is
> > apparently YUV 4:4:4, so the conversion is easy.
> 
> Uh, YUY2 is 4:2:2, not 4:4:4.
> 
> Also pretty sure we use Y'CbCr internally, not CrCb.

Sorry, didn't bother to look up the 4cc details.  As we all know, video format naming is a mess.... and confusing.

Comment 13

[firefoxiscute]

Attachment: nspr log with Nightly 27.0a1 (2013-10-03)

Fixed with the latest Nightly 27.0a1 (2013-10-03), as well as latest 25.0 beta.
I can still reproduce it with firefox 24.0.

Comment 14

[Randell Jesup [:jesup] (needinfo me)]

Those have long ago gotten to release.  Thanks