Oxide doesn't properly report the front/back position for enumerated video devices

Bug #1568145 reported by Alexandre Abreu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Oxide
Fix Released
High
Alexandre Abreu
1.15
Won't Fix
High
Unassigned

Bug Description

In Oxide's video device enumeration code using libhybris,

http://bazaar.launchpad.net/~oxide-developers/oxide/oxide.trunk/view/head:/shared/browser/media/oxide_video_capture_device_factory_linux.cc#L46

the device facing position is "encoded" as a string and passed down to the content layer when creating the VCD name:

http://bazaar.launchpad.net/~oxide-developers/oxide/oxide.trunk/view/head:/shared/browser/media/oxide_video_capture_device_factory_linux.cc#L82

This name is then used when creating the MEdiaStreamDevice that serves as a backend for the capture device

https://git.launchpad.net/~oxide-developers/oxide/+git/chromium/tree/content/browser/renderer_host/media/video_capture_manager.cc#n842

This then creates a MEdiaStreamDevice based on that positional name

https://git.launchpad.net/~oxide-developers/oxide/+git/chromium/tree/content/public/common/media_stream_request.cc#n29

but :

1. the code is #ifdef'd to Android

which get its info from

https://git.launchpad.net/~oxide-developers/oxide/+git/chromium/tree/media/capture/video/android/java/src/org/chromium/media/VideoCaptureAndroid.java#n58

2. the spelling used (in the android case) is lower case "front",

The result is that no matter what camera we are using we get an 'unspecified' as the camera position on the API side,

Changed in oxide:
assignee: nobody → Alexandre Abreu (abreu-alexandre)
importance: Undecided → High
status: New → In Progress
Changed in oxide:
milestone: none → branch-1.15
Changed in oxide:
milestone: branch-1.15 → branch-1.16
Changed in oxide:
milestone: branch-1.16 → branch-1.17
Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :
Changed in oxide:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.