Closed Bug 1827516 Opened 2 years ago Closed 2 years ago

Unable to change/disable video subtitles after playback started, subtitle menu closes before mouse pointer reaches it

Categories

(Core :: DOM: Core & HTML, defect)

x86_64
All
defect

Tracking

()

VERIFIED FIXED
114 Branch
Tracking Status
firefox-esr102 --- unaffected
firefox112 --- unaffected
firefox113 + verified
firefox114 + verified

People

(Reporter: jan, Assigned: emilio)

References

(Regression)

Details

(Keywords: nightly-community, regression)

Attachments

(4 files)

Attached video subtitle-screencast.webm (deleted) —

KDE Wayland, Debian Testing
I can't change/disable subtitles anymore. It's as if there is space between the CC button and the subtitle menu and then it closes when I unhover the CC button.

mozregression --good 2023-02-01 --bad 2023-04-11 -a https://brenopolanski.github.io/html5-video-webvtt-example/

9:43.09 INFO: Last good revision: be6ab6cd894d8c24ff38dca59c1820ef8c51a831
9:43.09 INFO: First bad revision: 4d6b19e4c8bdd48e2553678ca95add19f6a0eedf
9:43.09 INFO: Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=be6ab6cd894d8c24ff38dca59c1820ef8c51a831&tochange=4d6b19e4c8bdd48e2553678ca95add19f6a0eedf

4d6b19e4c8bdd48e2553678ca95add19f6a0eedf Emilio Cobos Álvarez — Bug 1825825 - Tweak DevTools walker to preserve behavior after the previous patches. r=devtools-reviewers,nchevobbe
0d254cc9e9c4a17c2d8a9af63bbac57e81cb84d9 Emilio Cobos Álvarez — Bug 1825825 - Simplify NAC setup. r=smaug

Flags: needinfo?(emilio)
Assignee: nobody → emilio
Attached video video-control-hover-broken.webm (deleted) —

Same regression range:
Hovering the area of the video control bar doesn't unhide it anymore.
Now, the mouse pointer must hover the video above the control bar to unhide the control bar.

Yep, I'm looking into this, I just had to fix the debugger first lol.

Depends on: 1827850
Duplicate of this bug: 1828332
OS: Linux → All
Status: NEW → ASSIGNED
Flags: needinfo?(emilio)

No behavior change here, but I was looking and this code can be simpler.

The continue in GetContentInThisDocument meant that if we call that in a
UA widget we'd return null...

This breaks the following check:

https://searchfox.org/mozilla-central/rev/31f5847a4494b3646edabbdd7ea39cb88509afe2/toolkit/content/widgets/videocontrols.js#1373-1381

Which was introduced in bug 1513600 to fix precisely this bug...

Depends on D175714

Comment on attachment 9328948 [details]
Bug 1827516 - Fix retargeting of UAWidget.elementFromPoint(). r=smaug

Beta/Release Uplift Approval Request

  • User impact if declined: See comment 0 and dupe
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: comment 0 and bug 1828332 show examples of this.
  • List of other uplifts needed: none
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Relatively straight-forward change.
  • String changes made/needed: none
  • Is Android affected?: Yes
Attachment #9328948 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/972e233cb181 Cleanup some event target parent code. r=smaug https://hg.mozilla.org/integration/autoland/rev/e2222a8b3390 Fix retargeting of UAWidget.elementFromPoint(). r=smaug
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
QA Whiteboard: [qa-triaged]
Status: RESOLVED → VERIFIED

Comment on attachment 9328948 [details]
Bug 1827516 - Fix retargeting of UAWidget.elementFromPoint(). r=smaug

Approved for 113.0b6.

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

Reproduced the issue with Firefox 114.0a1 (20230416211848) on Windows 10x64 by following STR from comment 0 and from bug 1828332. The subtitles cannot be changed and the video controls are not displayed when entering the video area from the bottom.
The issue is verified fixed with Firefox 114.0a1 (2023-04-19) and 113.0b6 (20230419174221) from comment 10 on Windows 10x64, macOS 12 and Ubuntu 22.04. The subtitles can be successfully changed and the video controls are displayed when entering the video area from the bottom or when changing the seek bar position.

QA Whiteboard: [qa-triaged]
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: