ubuntu-webapps-bugs team mailing list archive

[Alexandre Abreu <alexandre.abreu@xxxxxxxxxxxxx>]

Public bug reported:

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,

** Affects: oxide
     Importance: High
     Assignee: Alexandre Abreu (abreu-alexandre)
         Status: In Progress

** Changed in: oxide
     Assignee: (unassigned) => Alexandre Abreu (abreu-alexandre)

** Changed in: oxide
   Importance: Undecided => High

** Changed in: oxide
       Status: New => In Progress

-- 
You received this bug notification because you are a member of Ubuntu
WebApps bug tracking, which is subscribed to Oxide.
https://bugs.launchpad.net/bugs/1568145

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

Status in Oxide:
  In Progress

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,

To manage notifications about this bug go to:
https://bugs.launchpad.net/oxide/+bug/1568145/+subscriptions