Closed Bug 1768201 Opened 2 years ago Closed 2 years ago

The Fast Forward/Backward commands are unavailable in the PiP window on Netflix

Categories

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

Desktop
All
defect
Points:
3

Tracking

()

VERIFIED FIXED
102 Branch
Tracking Status
relnote-firefox --- 100+
firefox-esr91 --- unaffected
firefox100 + verified
firefox101 --- verified
firefox102 --- verified

People

(Reporter: ada.cristea, Assigned: kpatenio)

References

Details

(Keywords: regression, Whiteboard: [fidefe-MR1-2022])

Attachments

(1 file)

Note

  • When the user launches the PiP for the first time in a fresh profile on Netflix, it will display at the very bottom-right edge and the user can forward/backward the video by tapping on the right/left keyboard arrows, he will notice that the video is not being skipped forward/backward.

Affected versions

  • Nightly v102.0a1
  • Beta v101.0b2
  • Release

Affected platforms

  • Windows 10 x64
  • Mac OS 11.6 (Big Sur)
  • Ubuntu 20.04

Steps to reproduce

  1. Launch browser in a new profile.
  2. Play any video.
  3. Launch the PiP window.
  4. Tap the right/left arrows on the keyboard.

Expected result

  • The user is able to seek forward/backward the video.

Actual result

  • The used is unable to seek forward/backward the video.

Regression range

  • This appears to be a regression, however it can't be investigated while the Widevine issue still reproduces. (1767953)

Hmm. I would be surprised if this were a regression; my expectation would be that these buttons never worked, as the pause button didn't until we manually patched it in bug 1752132.

Currently we're running fallback functions when we fast forward/go backwards with left or right arrow keys (searchfox). We may need to leverage the Netflix player API like we did for the pause/play button.

(In reply to kpatenio from comment #2)

Currently we're running fallback functions when we fast forward/go backwards with left or right arrow keys (searchfox). We may need to leverage the Netflix player API like we did for the pause/play button.

Do you happen to have any availability to look into that angle? If not I'll see what I can do.

Flags: needinfo?(kpatenio)

(In reply to Molly Howell (she/her) [:mhowell] from comment #3)

(In reply to kpatenio from comment #2)

Currently we're running fallback functions when we fast forward/go backwards with left or right arrow keys (searchfox). We may need to leverage the Netflix player API like we did for the pause/play button.

Do you happen to have any availability to look into that angle? If not I'll see what I can do.

I have availability at the moment. I can take an initial look at the problem.

Flags: needinfo?(kpatenio)
Assignee: nobody → kpatenio
Status: NEW → ASSIGNED
Summary: The Fast Forward/Backward commands are unavailable in the small PiP window on Netflix → The Fast Forward/Backward commands are unavailable in the PiP window on Netflix
Has STR: --- → yes
Pushed by kpatenio@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9f13cd65b345
add seek forward and backward key shortcuts for pip on netflix. r=pip-reviewers,mhowell

Comment on attachment 9275633 [details]
Bug 1768201 - add seek forward and backward key shortcuts for pip on netflix. r=#pip-reviewers!

Beta/Release Uplift Approval Request

  • User impact if declined: Firefox users that have PiP enabled for a Netflix video won't be able to seek backwards/forward in the video's timeline using the left or right arrow keyboard shortcuts. This is inconsistent with other major sites like YouTube and Prime Video, where the left/right arrow shortcuts work as expected. To improve PiP usability for all PiP users, we should have this fix merged in as soon as possible.
  • 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: 1. Navigate to Netflix
  1. Watch a video
  2. Enable Picture in Picture mode via PiP toggle or context menu (after right clicking) on the video
  3. Click on the pip window that appears on screen
  4. Test keyboard shortcuts: press left arrow key to go backwards, right arrow key to move forwards, home button to go to the start of the video, end button to go to the end of the video
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): The fix only affects Netflix. Even if the fix fails to work for some reason, it will not break Picture in Picture as a whole; the shortcuts would simply not work in that case. (Regarding automated tests: we do not have automated tests specifically for Netflix, since we do not have the capability to test site specific video wrappers with live sites right now. However, we already have tests that verify overall keyboard shortcut functionality for PiP).
  • String changes made/needed: None
  • Is Android affected?: No
Attachment #9275633 - Flags: approval-mozilla-release?
Attachment #9275633 - Flags: approval-mozilla-beta?
Flags: qe-verify+
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
QA Whiteboard: [qa-triaged]

Comment on attachment 9275633 [details]
Bug 1768201 - add seek forward and backward key shortcuts for pip on netflix. r=#pip-reviewers!

Approved for 101.0b5.

Attachment #9275633 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Points: --- → 3

This fix was verified in Nightly v102.0a1 from 2022-05-11, Beta v101.0b5 on Win10, Mac OS 11 and Ubuntu 22.

Status: RESOLVED → VERIFIED
Flags: qe-verify+

Comment on attachment 9275633 [details]
Bug 1768201 - add seek forward and backward key shortcuts for pip on netflix. r=#pip-reviewers!

Approved for 100.0.1

Attachment #9275633 - Flags: approval-mozilla-release? → approval-mozilla-release+

Reverted only the changes to PictureInPictureChild.jsm in mozilla-release for causing eslint failures
https://hg.mozilla.org/releases/mozilla-release/rev/b9f6183616e104d672e34b00c07e5603d4496133

This fix has also been verified on Release v100.0.1 on Windows 10, Mac OS 11.6.5, Ubuntu 20.04 LTS (X11 Window Protocol), Ubuntu 22.04 LTS (Wayland Window Protocol) and Windows 7. The right and Left arrow buttons seek forward and back while in PiP.

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

Attachment

General

Creator:
Created:
Updated:
Size: