Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1678453 - Firefox Wayland By Default On Gnome
Summary: Firefox Wayland By Default On Gnome
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Changes Tracking
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Martin Stransky
QA Contact:
URL:
Whiteboard:
Depends On: ffwayland
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-02-18 19:48 UTC by Ben Cotton
Modified: 2019-07-11 11:04 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-07-11 11:04:41 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ben Cotton 2019-02-18 19:48:58 UTC
This is a tracking bug for Change: Firefox Wayland By Default On Gnome
For more details, see: https://fedoraproject.org/wiki/Changes/Firefox_Wayland_By_Default_On_Gnome

Firefox is going to run natively on Gnome Wayland session and won't use XWayland/X11 Gtk+ backend by default. The XWayland/X11 Gtk+ backend can be still used via additional firefox-x11 package. This change affects Gnome only and won't be enabled for other Wayland compositors (KDE Plasma, Sway).

Comment 1 Martin Stransky 2019-02-19 10:59:20 UTC
Wayland bugs are tracked at Bug 1054334

Comment 2 Ben Cotton 2019-02-19 20:30:21 UTC
According to the Fedora 30 schedule[1], today is the deadline for changes to be in a testable state. If your change is ready to be tested, please set the status to MODIFIED. If you know your change will not be ready for Fedora 30, you can set the version to rawhide and notify bcotton. For more information about this milestone, see the Changes Policy[2].

[1] https://fedoraproject.org/wiki/Releases/30/Schedule
[2] https://fedoraproject.org/wiki/Changes/Policy#Change_Checkpoint:_Completion_deadline

Comment 3 Martin Stransky 2019-02-20 08:10:06 UTC
Yes, the default Wayland backend is already there, although there isn't the latest package available due to gcc9 build failures (bah). Also new Firefox 66 with more wayland fixes updates comes to Fedora on 2019-03-19 [1] so the go/no-go decision should be made after FF66 release.

[1] https://wiki.mozilla.org/Release_Management/Calendar

Comment 4 Robert Mader 2019-03-04 23:34:44 UTC
For the final decision I'd like to point out a GS bug [1] which affects window screenshots, animations and screencasting of FF on Wayland. I made an attempt to fix that [2][3], but it turned out that bigger changes will be required. That work[4][5] will unfortunately not make it into GS/Mutter 3.32 :(

[1] https://gitlab.gnome.org/GNOME/mutter/issues/146
[2] https://gitlab.gnome.org/GNOME/mutter/merge_requests/384
[3] https://gitlab.gnome.org/GNOME/gnome-shell/merge_requests/341
[4] https://gitlab.gnome.org/GNOME/mutter/merge_requests/409
[5] https://gitlab.gnome.org/GNOME/mutter/tree/gbsneto/content-part2

Comment 5 Ben Cotton 2019-03-05 21:49:57 UTC
We have reached the Code Complete (100%) milestone in the Fedora 30 development cycle. At this point, all Changes should be fully code complete and ready for testing during the beta freeze. If your Change has reached this milestone, please set the status to ON_QA. If it has not, this Change will be submitted to FESCo to evaluate the contigency plan and decide if the Change will continue in the Fedora 30 cycle.

Comment 6 Martin Stransky 2019-03-06 09:43:35 UTC
The target Firefox version is Firefox 66 here (Fedora 30 has 64 due to recent build failures). According to https://wiki.mozilla.org/Release_Management/Calendar Firefox 66 is going to be released on 2019-03-19 so this will be testable after that.

Comment 7 Fedora Blocker Bugs Application 2019-03-11 15:20:44 UTC
Proposed as a Freeze Exception for 30-beta by Fedora user sgallagh using the blocker tracking app because:

 FESCo would like to see this Change land if it would not delay the Beta release. The standard blocker/exception review will make the final call. If it does not land in F30 Beta, this Change is Deferred to F31.

Comment 8 Geoffrey Marr 2019-03-11 20:28:21 UTC
Discussed during the 2019-03-11 blocker review meeting: [1]

The decision to classify this bug as an "AcceptedFreezeException" was made as, while we are concerned about the gnome-shell issue and the late ff66 release date here, it makes no sense to deny an FE for those reasons, but take the Change post-Beta. So we grant it an FE, but will ask FESCo if it still thinks the change should land in F30 at all given those considerations.

[1] https://meetbot.fedoraproject.org/fedora-blocker-review/2019-03-11/f30-blocker-review.2019-03-11-16.04.txt

Comment 9 Adam Williamson 2019-03-11 22:30:59 UTC
For the record, it was pointed out that so far as screenshots are confirmed, the bug discussed above affects taking single-window screenshots (alt+print screen) but not a whole desktop screenshot (print screen). This does make it less serious. I'm not sure yet precisely what the impact on screencasting and animations is.

Comment 10 Adam Williamson 2019-03-11 22:35:24 UTC
https://pagure.io/fesco/issue/2071#comment-559596

Comment 11 Robert Mader 2019-03-12 11:19:55 UTC
Concerning screencasting: it should only affect window-screencasting, like with screenshots. AFAIK this is new functionality in GS Wayland session (was not available in Fedora 29/GS 3.30), so it's at least not a regression.

About animations: from my experience it makes certain animation, e.g. maximizing, a bit glitchy. On the other hand, switching to the Wayland backend makes other things less glitchy (starting FF with accelerated layers). So I'd say it's a trade-off.

Comment 12 Martin Stransky 2019-03-12 11:39:50 UTC
First Firefox 66 builds head to koji now. I expect them in testable state later this week.

Comment 13 Adam Williamson 2019-03-18 19:40:07 UTC
FESCo voted to defer this Change to F31, so I am removing it from the F30 Beta FE list (it only gets an F30 Beta FE if it's an F30 Change, obviously).

Comment 14 Martin Stransky 2019-03-21 08:45:51 UTC
The change is reverted for Fedora 30 in the firefox 66.0 update:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-38ec450c27

Comment 15 Martin Stransky 2019-03-22 07:24:03 UTC
There's a new update for that: https://bodhi.fedoraproject.org/updates/FEDORA-2019-2e62c6961a

Comment 16 Adam Williamson 2019-03-22 15:29:28 UTC
Um. Wait. Are you saying that the Firefox in F30 stable *right now* is Wayland-native? So if we want to have X11-native Firefox for Beta we have to pull in that update?

Comment 17 Adam Williamson 2019-03-22 15:30:16 UTC
To be clear, I was previously reading your comment #6 as saying that Wayland-native Firefox was not going to be enabled for F30 until that Firefox 66 release landed.

Comment 18 Martin Stransky 2019-03-22 16:04:48 UTC
(In reply to Adam Williamson from comment #16)
> Um. Wait. Are you saying that the Firefox in F30 stable *right now* is
> Wayland-native? So if we want to have X11-native Firefox for Beta we have to
> pull in that update?

Yes, that correct. The update is here:
https://bodhi.fedoraproject.org/updates/FEDORA-2019-2e62c6961a

Also new firefox 66.0.1 with security updates from Pwn2Own is coming out on Monday.

(In reply to Adam Williamson from comment #17)
> To be clear, I was previously reading your comment #6 as saying that
> Wayland-native Firefox was not going to be enabled for F30 until that
> Firefox 66 release landed.

I was meaning in comment #6 that the Firefox 66 build will have all important Wayland fixes
and will be ready for testing/final decision.

The default Wayland backend has been enabled on Firefox / F30 since this bug was created.

Comment 19 Martin Stransky 2019-03-22 16:12:13 UTC
Btw. also see comment 3 where I stated the default Wayland is already there and switched the bug to modified.

Comment 20 Adam Williamson 2019-03-22 16:26:41 UTC
Ah. OK. In that case we'll need an FE bug. I'll file one...

Comment 21 Adam Williamson 2019-03-22 16:32:19 UTC
https://bugzilla.redhat.com/show_bug.cgi?id=1691852 filed. So you think we should wait till Monday and pull in the new pwn2own fixes for Beta?

Comment 22 Martin Stransky 2019-03-22 16:35:35 UTC
(In reply to Adam Williamson from comment #21)
> https://bugzilla.redhat.com/show_bug.cgi?id=1691852 filed. So you think we
> should wait till Monday and pull in the new pwn2own fixes for Beta?

Yes, I'm submitting the builds now.

Comment 23 Adam Williamson 2019-03-22 16:58:53 UTC
Uh...but what do you mean about Monday, then? :) If the builds you're submitting right now have the fixes, they should be done later today and we can pull them into a compose tonight or tomorrow...

Comment 24 Martin Stransky 2019-03-22 17:12:46 UTC
(In reply to Adam Williamson from comment #23)
> Uh...but what do you mean about Monday, then? :) If the builds you're
> submitting right now have the fixes, they should be done later today and we
> can pull them into a compose tonight or tomorrow...

Sure, grab them when they're ready, it's firefox-66.0.1-1.

Comment 25 Adam Williamson 2019-03-22 17:28:43 UTC
OK, so the build should complete in about 8 hours, we will go ahead and pull it into a new compose at that point. Thanks.

Comment 26 Martin Stransky 2019-07-11 11:04:41 UTC
Fixed in Rawhide.


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