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

Bug #1568145 reported by Alexandre Abreu on 2016-04-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Oxide
High
Alexandre Abreu
1.15
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
Changed in oxide:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers