Closed Bug 1581806 Opened 5 years ago Closed 5 years ago

devicechange event is triggered for any device in windows

Categories

(Core :: WebRTC: Audio/Video, defect, P2)

defect

Tracking

()

VERIFIED FIXED
mozilla71
Tracking Status
firefox70 --- verified
firefox71 --- verified

People

(Reporter: achronop, Assigned: achronop)

References

(Depends on 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

devicechange notifications are triggered with any device change, not just video and audio devices. For instance, plugging/unplugging a USB flash disk creates a device change notification.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7e694f22c762cd75133ebb7566f21474f9bd8834
Priority: -- → P2

After Bug 1581193 devicechange notifications were triggered with any device change, not just video and audio devices. This patch limits down the notifications to only video and audio input devices change.

Blocks: 1572281
Pushed by achronopoulos@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1feec83ee6f9
Trigger devicechange event for audio and video input devices only. r=dminor
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71

Please request Beta approval on this.

Flags: needinfo?(achronop)

Comment on attachment 9093285 [details]
Bug 1581806 - Trigger devicechange event for audio and video input devices only. r?dminor

Beta/Release Uplift Approval Request

  • User impact if declined: devicechange notifications are triggered with any device change, not just video and audio devices. For instance, plugging/unplugging a USB flash disk creates a device change notification.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: Steps are mentioned in https://bugzilla.mozilla.org/show_bug.cgi?id=1581193#c0
  • List of other uplifts needed: Bug 1581193
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is low risk because it does not alter much the way that it already works, it only filters out the non-media devices.
  • String changes made/needed: N/A
Flags: needinfo?(achronop)
Attachment #9093285 - Flags: approval-mozilla-beta?
Flags: qe-verify+
QA Whiteboard: [qa-triaged]

Hi Alex, I tried to reproduce the issue on Firefox Nightly 71.0a1 (2019-09-19 and respectively 2019-09-22) under Ubuntu 18.04 (x64) with a usb flash disk, but couldn't trigger the "device change!" message at all. Are there some extra steps that I should follow or something else to do? Thanks!

Flags: needinfo?(achronop)

This is landed in 2019-09-18 so it should be fixed in 2019-09-19 and you wont be able to reproduce it after that. Try the Nightly from 2019-09-18 to reproduce it. Also, I was using fedora but I don't think it affects that scenario.

Flags: needinfo?(achronop)

Comment on attachment 9093285 [details]
Bug 1581806 - Trigger devicechange event for audio and video input devices only. r?dminor

Part of the fix for regression in 1581193. OK for uplift for beta 10.

Attachment #9093285 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

I successfully reproduced the issue on Firefox Nightly 71.0a1 (2019-09-18) under Windows 10 x64 using a usb keyboard device (thanks for the infos Alex).

The issue is no longer reproducible on latest Nightly 71.0a1 (2019-09-24) and Firefox Beta 70.0b9. Tests were performed on Windows 10 (x64), Ubuntu 16.04 (x64) and macOS 10.14.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: