Closed Bug 1584492 Opened 2 years ago Closed 2 years ago

Set and propagate PIP window flag to nsWindow


(Core :: Widget: Gtk, enhancement, P3)




Tracking Status
firefox71 --- fixed


(Reporter: stransky, Assigned: stransky)




(2 files)

PIP Window needs a special config on Linux so we need to provide that information to toolkit. Add it to nsWidgetInitData which propagates it to nsWindow. Special PIP window config on Linux may include:

  • open window manager menu on right mouse click to place it on
    top/all workspaces (recently it's unused).
  • don't use alpha blending/compositing for faster rendering
  • disable titlebar shapes on X11, the window should not have round corners
  • use direct rendering for video on Wayland by dmabuf surfaces
Priority: -- → P3

Pushed by
Add mPIPWindow to nsWidgetInitData and set it for PIP window on Linux/Gtk, r=mats

Keywords: checkin-needed

Backed out changeset 7eb6fa2d58d0 (Bug 1584492) because D47410 failed to land

Backout link:

Got this when trying to land D47410:
applying /tmp/tmpn0_bV7 1 out of 1 hunk ignored -- saving rejects to file xpfe/appshell/nsAppShellService.cpp.rej 2 out of 2 hunks ignored -- saving rejects to file widget/nsWidgetInitData.h.rej abort: patch command failed: exited with status 256

Similar message when i imported the patch to manually land this.

Also clearing checkin-needed on bug 1583852 and bug 1536747 because they need these changes.

Flags: needinfo?(stransky)
Flags: needinfo?(stransky)
Keywords: checkin-needed

Pushed by
Add mPIPWindow to nsWidgetInitData and set it for PIP window on Linux/Gtk, r=mats
[Linux] Add mIsPIPWindow to nsWindow, r=jhorak

Keywords: checkin-needed
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
You need to log in before you can comment on or make changes to this bug.