Closed Bug 1757931 Opened 2 years ago Closed 2 years ago

Grabbing the PiP by the edges in a Linux + Wayland environment, will not allow enlargement, only shrinking

Categories

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

Desktop
Linux
defect

Tracking

()

RESOLVED DUPLICATE of bug 1739622
Tracking Status
firefox-esr91 --- wontfix
firefox97 --- wontfix
firefox98 --- wontfix
firefox99 --- wontfix
firefox100 --- wontfix

People

(Reporter: danibodea, Unassigned)

References

Details

(Whiteboard: [fidefe-MR1-2022])

Attachments

(2 files)

Note

  • When the user loads a video in a Linux + Wayland environment, lunches in PiP and grabs any edge, he will notice that a blank PiP window WILL NOT enlarge, only shrink.

Affected versions

  • Nightly v99.0a1
  • ESR v91.6.0esr

Affected platforms

  • Ubuntu 21 + Wayland
  • This issue does not occur on a Ubuntu 21 + xwayland

Steps to reproduce

  1. Launch browser.
  2. Load a video
  3. Click the PiP button
  4. Grab any of the edges and attempt to enlarge the PiP window.

Expected result

  • The PiP window should enlarge.

Actual result

  • The PiP window WILL NOT enlarge.

Regression range

  • Not a recent regression.

Additional notes
*

Whiteboard: [fidefe-MR1-2022]
Priority: -- → P2
Priority: P2 → P3

I can try to repro this but that's a bit surprising since I'm pretty sure i can shrink and grow the PiP window just fine on fedora at least.

Flags: needinfo?(daniel.bodea)

Err sorry that was meant for me

Flags: needinfo?(daniel.bodea) → needinfo?(emilio)

I couldn't repro this in neither Fedora nor Ubuntu 21, can you try to see if it's a regression or something, or if there's something else that might be needed to reproduce this? Thanks.

Flags: needinfo?(emilio) → needinfo?(daniel.bodea)

You should make sure Ubuntu 20/21/22 is updated to the latest version. It should install the "Wayland" Window Protocol by default.
Furthermore, Nightly is now set to open in Wayland Window Protocol by default, as well.

For the issue to reproduce, you'd have to make sure that the value of the "Window Protocol" attribute from "about:support" is "wayland" and not "x11", "xwayland" or others.

To force open the browser is Wayland Window Protocol, you can take these steps:

  1. Open a terminal window inside the browser folder.
  2. type "export MOZ_ENABLE_WAYLAND=1"
  3. then type "./firefox"
    The browser should open in Wayland Window Protocol.

P.S. The issue still reproduces in the latest Nightly.

Flags: needinfo?(daniel.bodea)
Flags: needinfo?(emilio)

I know how to get wayland enabled (I use wayland regularly) but still I can't reproduce?

Dani, can you try to see if this is a regression on your machine or so?

Flags: needinfo?(emilio) → needinfo?(daniel.bodea)

This issue occurs on all channels (as previously mentioned). Upon attempting to bring more information to the table, it appears that the following versions cannot be tested because the content process crashes (any new tab crashes): v50.0a1, v66.0a1, v71.0a1, v83.0a1, v86.0a1. These versions have to be forced open with wayland. Version v96.0a1 reproduces the issue as described in comment 0. This bug is not a regression and it may have a connection to the hardware configuration.

I have attached the about:support information of the latest Nightly on the affected machine.
Let me know if there is more I can help with.

Flags: needinfo?(daniel.bodea) → needinfo?(emilio)

Can you run mozregression with MOZ_ENABLE_WAYLAND=1 MOZ_DISABLE_CONTENT_SANDBOX=1? That should avoid the tab crashes, with that I can run at least as far as FF48 (probably more).

Flags: needinfo?(emilio) → needinfo?(daniel.bodea)

I wish I could help, but the mozregression-gui version does not seem to work in Wayland and the mozregression CLI version could not be installed.
I could provide with a regressor investigation if you could help me with some steps to install/run mozregression in Wayland.

Flags: needinfo?(daniel.bodea) → needinfo?(emilio)

(In reply to Bodea Daniel [:danibodea] from comment #8)

I wish I could help, but the mozregression-gui version does not seem to work in Wayland and the mozregression CLI version could not be installed.

How not? On a terminal, you can:

$ pip3 install --user mozregression

And then you should be able to run mozregression. If that's failing, I need a more specific error to help, how is it failing?

Flags: needinfo?(emilio) → needinfo?(daniel.bodea)

I've tried the command you've specified in one of the comments above, it appeared as if it was installing, but mozregression still does not work. Waiting for your opinion.

Flags: needinfo?(daniel.bodea) → needinfo?(emilio)

This issue reproduces on 3 different systems that have been tested so far. It is not specific to 1 system.

It seems you don't have ~/.local/bin in your PATH, does using ~/.local/bin/mozregression rather than just mozregression work?

Flags: needinfo?(emilio) → needinfo?(daniel.bodea)

Yes! It appears it does.

Steps to re-attempt investigation for a regressor:

  1. Open the terminal
  2. type: export MOZ_ENABLE_WAYLAND=1
  3. type: export MOZ_DISABLE_CONTENT_SANDBOX=1
  4. type: ~/.local/bin/mozregression --launch 59
  • v60.0a1 - can't be tested since it does not have PiP implemented, and neither is wayland.
  1. type: ~/.local/bin/mozregression --launch 69
  • v70.0a1 - wayland is implemented, but PiP isn't.
  1. type: ~/.local/bin/mozregression --launch 79
  • v80.0a1 - "wayland/drm" implemented and PiP also in it's early stages; the issue reproduces!
  1. Starting a bisection between the last 2 versions above.
    type: ~/.local/bin/mozregression --good 69 --bad 79
  • v70.0a1 - wayland is implemented, but PiP isn't. - setting result as GOOD.
  • v80.0a1 - "wayland/drm" implemented and PiP also in it's early stages; the issue reproduces! - setting result as BAD.
  • v71.0a1 - wayland is implemented, but PiP isn't. - setting result as GOOD.
  • v72.0a1 - wayland and PiP are implemented; the issue reproduces! - setting result as BAD.
  • 72.0a1 (20191029212748) - wayland is implemented, PiP toggle is displayed but isn't working at all. - setting result as GOOD to get e newer build where PiP works.
  • 72.0a1 (20191109093718) - wayland and PiP are implemented, the issue reproduces!

Based on the testing results detailed above, this bug is not a regressor but is reproducing since its implementation.
Need Info me if further investigation is needed.

Flags: needinfo?(daniel.bodea)
Flags: needinfo?(emilio)

Ah, d'oh, indeed I can reproduce, but I misinterpreted your report. I can resize it from the corners. Let's dupe the other way around since bug 1739622 was older.

Flags: needinfo?(emilio)
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: