Closed Bug 1804895 Opened 1 year ago Closed 1 year ago

[Hulu] The key shortcut for mute/unmute a PiP video is not working accordingly

Categories

(Toolkit :: Picture-in-Picture, defect)

Desktop
All
defect

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox-esr102 --- wontfix
firefox107 --- wontfix
firefox108 --- wontfix
firefox109 --- wontfix
firefox110 --- verified

People

(Reporter: ailea, Assigned: niklas)

References

()

Details

Attachments

(1 file)

Found in

  • 109.0a1

Affected versions

  • 107.0.1, 108.0, 109.0a1 (2022-12-09)

Tested platforms

  • Affected platforms: Windows 10, macOS 12, Ubuntu 20.

Steps to reproduce

  1. Access the following website: https://www.hulu.com/
  2. Play any video and open the PiP window.
  3. Press ctrl + up arrow twice or ctrl + down arrow twice.

Expected result

  • ctrl + up arrow should unmute the PiP video and ctrl + down arrow should mute the PiP video (this is what the mute/unmute pip icon tooltip says and this is the behavior on e.g youtube). ctrl + up arrow should not mute the video and ctrl + down arrow should not unmute the video.

Actual result

  • Both combinations (ctrl + up arrow and ctrl + down arrow) can mute and unmute the PiP video.

Regression range

  • Not a regression.
Severity: -- → S4
Assignee: nobody → nbaumgardner
Status: NEW → ASSIGNED
Pushed by nbaumgardner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fa3c02de56dc
Update setMuted method for Hulu video wrapper to only toggle when setting a new muted state. r=pip-reviewers,mhowell
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch

The patch landed in nightly and beta is affected.
:niklas, is this bug important enough to require an uplift?

  • If yes, please nominate the patch for beta approval.
  • If no, please set status-firefox109 to wontfix.

For more information, please visit auto_nag documentation.

Flags: needinfo?(nbaumgardner)

Verified - Fixed in latest Nightly 110.0a1 (2022-12-16) using Windows 10, macOS 12 and Ubuntu 20.

Flags: needinfo?(nbaumgardner)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: