ubuntu-webapps-bugs team mailing list archive

[Chris Coulson <chrisccoulson@xxxxxxxxxx>]

** Changed in: oxide
       Status: In Progress => Fix Released

-- 
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:
  Fix Released
Status in Oxide 1.15 series:
  Won't Fix

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