Closed Bug 1700674 Opened 5 years ago Closed 4 years ago

Picture-in-picture window doesn't use native fullscreen even if full-screen-api.macos-native-full-screen is set

Categories

(Core :: Widget: Cocoa, defect, P2)

All
macOS
defect

Tracking

()

VERIFIED FIXED
103 Branch
Tracking Status
firefox103 --- verified

People

(Reporter: mstange, Assigned: KrisWright)

References

Details

(Whiteboard: [mac:fullscreen:native-affected])

Attachments

(1 file)

Steps to reproduce:

  1. Be on macOS, and set full-screen-api.macos-native-full-screen to true.
  2. Go to a YouTube video.
  3. Open the video in PiP mode.
  4. Double-click the PiP window.

Expected results:
The video should be displayed in native fullscreen.

Actual results:
The video is displayed in non-native fullscreen.

Severity: -- → S2
Priority: -- → P2
Whiteboard: [mac:fullscreen] → [mac:fullscreen:native-affected]

The problem seems to be happening right here, where we aren't getting a native fullscreen support. Because of this it displays in non-native fullscreen even though the pref is flipped.

This adds the macos native fullscreen attribute to picture-in-picture windows so that when the pref is enabled, we view native fullscreen.

Assignee: nobody → kwright
Status: NEW → ASSIGNED
Pushed by kwright@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/709f48d5b837 Add native full screen attribute to picture-in-picture r=pip-reviewers,mhowell
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 103 Branch
Flags: qe-verify+

I managed to reproduce this issue on Firefox 102.0(build ID: 20220623063721) on macOS 11. Verified as fixed on Firefox 103.0b2(build ID: 20220628190840) and Nightly 104.0a1(build ID: 20220629215821) on macOS 11.

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

Attachment

General

Created:
Updated:
Size: