Closed Bug 1695453 Opened 3 years ago Closed 3 years ago

Rename IsWaylandDisabled to IsWaylandEnabled

Categories

(Core :: Graphics, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: rmader, Assigned: rmader)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

To make things easier to read.

Inversed logic has been proven to be more difficult to read,
so use the simple positive variant.

Also add a simple sanity check for WAYLAND_DISPLAY so if people
set MOZ_ENABLE_WAYLAND in a X11 session don't get undesired behavior.

While on it, change a check for XDG_SESSION_TYPE to also use
WAYLAND_DISPLAY, improving behavior when launching FF from a TTY
or a TTY-launched session (e.g. via weston-launch).

WAYLAND_DISPLAY and DISPLAY are not expected to be set if
no Wayland or X11 server is available, so using them makes us behave
more predictable.

Assignee: nobody → robert.mader
Status: NEW → ASSIGNED
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/49a6125392f5
Rename IsWaylandDisabled to IsWaylandEnabled, r=stransky
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
Regressions: 1696319
Regressions: 1709382
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: