Closed Bug 1753401 Opened 4 months ago Closed 4 months ago

PiP not working on Hulu

Categories

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

Desktop
All
defect

Tracking

()

VERIFIED FIXED
99 Branch
Tracking Status
relnote-firefox --- 97+
firefox-esr91 --- unaffected
firefox96 --- unaffected
firefox97 + verified
firefox98 + verified
firefox99 --- verified

People

(Reporter: csasca, Assigned: mtigley)

References

(Depends on 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Affected versions

  • Firefox 97.0
  • Firefox 98.0a1

Affected platforms

  • Windows 7
  • macOS 11.6.3
  • Ubuntu 20.04

Steps to reproduce

  1. Launch Firefox
  2. Access Hulu and login
  3. Select any video and play it
  4. Click on PiP icon

Expected result

  • The video is played in PiP mode

Actual result

  • The video is paused

Regression range

  • Will see for a regression

Additional notes

  • The issue can be seen in the following attachment
  • PiP will work only if the video is minimized first from the player controls
Has Regression Range: --- → no
Has STR: --- → yes

Searched for regression and it seems it was introduced between 2022-01-03 and 04.
Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=85a1717fd579a0dbbf5e16a08dcbeee5a64f77fd&tochange=ba0ff402e454c72ede8868c249eabfd9d47e68ba

Bug 1742585 - Add PiP toggle visibility threshold preference

Has Regression Range: no → yes
Flags: needinfo?(rhopkinsdev)
Regressed by: 1742585
Flags: needinfo?(rhopkinsdev) → needinfo?(mtigley)

:csasca, is the issue fixed if you change the media.videocontrols.picture-in-picture.video-toggle.visibility-threshold preference to 1.0?

If that's the case, then this might require for Hulu to have its own PiP override, similar to what we do for youtube. Or we may need to increase that preference value back to 1.0.

Flags: needinfo?(mtigley)

Hey Micah, tried changing the pref to 1.0 and it seems to have resolved the issue on latest nightly 98.0a1.

Hardware: All → Desktop
Severity: S4 → S3
Flags: needinfo?(mtigley)
Assignee: nobody → mtigley
Status: NEW → ASSIGNED

Comment on attachment 9262947 [details]
Bug 1753401 - Change PiP toggle's visibility threshold back to 1.0. r=mhowell!

Beta/Release Uplift Approval Request

  • User impact if declined: Users won't be able to use the PiP toggle button on Hulu.
  • 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: See comment 0
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is a small adjustment to the existing pref
  • String changes made/needed:
Flags: needinfo?(mtigley)
Attachment #9262947 - Flags: approval-mozilla-release?
Attachment #9262947 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Pushed by mtigley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/70ea492ad245
Change PiP toggle's visibility threshold back to 1.0. r=mhowell
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch

Comment on attachment 9262947 [details]
Bug 1753401 - Change PiP toggle's visibility threshold back to 1.0. r=mhowell!

Uplift approved for 98 beta 3, thanks.

Attachment #9262947 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [qa-triaged]

This issue is reproducible on Firefox 97.0, Firefox 98.0b2 and Nightly 98.0a1 on Linux(Ubuntu 20.04). Fixed on Nightly 99.0a1 and the upcoming 98.0b3( downloaded from treeherder) on all platforms.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Comment on attachment 9262947 [details]
Bug 1753401 - Change PiP toggle's visibility threshold back to 1.0. r=mhowell!

Approved for 97.0.1. Do we have a bug for making this a site-specific workaround for the future?

Flags: needinfo?(mtigley)
Attachment #9262947 - Flags: approval-mozilla-release? → approval-mozilla-release+

Yes, we've got a bug on file in bug 1754604

Depends on: 1754604
Flags: needinfo?(mtigley)

Added to the 97.0.1 release notes.

Fixed an issue which led to Picture-in-Picture mode being unable to be toggled on Hulu

Verified on Firefox 97.0.1(20220216172458) on macOS Big Sur 11, Linux(Ubuntu 20.04) and Win7. PiP is working now.

You need to log in before you can comment on or make changes to this bug.