getSettings() should include deviceId

NEW
Unassigned

Status

()

Core
WebRTC: Audio/Video
P3
normal
2 months ago
a month ago

People

(Reporter: jib, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox57 affected)

Details

Forgot to include deviceId when we added getSettings() in bug 1213517.

STRs:
 1. Open https://jsfiddle.net/jib1/3jrkdc1q/ and share cam+mic

Expected result:

{
  "autoGainControl": false,
  "channelCount": 4,
  "echoCancellation": true,
  "noiseSuppression": true
  "deviceId": "hLSmbVie34mQOuLjqIdBxsaBoK+W0eESXBzeK+78guI="
}
{
  "frameRate": 30,
  "height": 480,
  "width": 640,
  "deviceId": "mSUcE4/pfvfWrrUCKPVgPsqTFtztaQ6T32O9yjs8SOs="
}

Actual result:

{
  "autoGainControl": false,
  "channelCount": 4,
  "echoCancellation": true,
  "noiseSuppression": true
}
{
  "frameRate": 30,
  "height": 480,
  "width": 640
}
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.