Closed Bug 1460255 Opened 2 years ago Closed 2 years ago

Allow to build both GLX and EGL OpenGL backends for X11/Wayland builds

Categories

(Core :: Graphics, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: stransky, Assigned: stransky)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 2 obsolete files)

This is a follow up from Bug 788319 comment 47

In order to make Wayland builds usable, implement run-time GL context switch to use GLX on X11 and EGL on Wayland.
Summary: Allow to build both GLX and EGL OpenGL backends for X11&Wayland builds → Allow to build both GLX and EGL OpenGL backends for X11/Wayland builds
Attached patch WIP patch (obsolete) — Splinter Review
Jeff, what do you think about this approach? It will leave X11/GLX backend as is and we can fix/update EGL for Wayland backend only.
Attachment #8974378 - Flags: feedback?(jgilbert)
Blocks: wayland
Assignee: nobody → stransky
Blocks: 1438144
Priority: -- → P2
Attachment #8974378 - Attachment is obsolete: true
Attachment #8974378 - Flags: feedback?(jgilbert)
Attachment #8975816 - Flags: review?(jmuizelaar) → review?(jgilbert)
Attachment #8975817 - Flags: review?(jmuizelaar) → review?(jgilbert)
Attachment #8975818 - Flags: review?(jmuizelaar) → review?(jgilbert)
Depends on: 1464037
Attachment #8975818 - Attachment is obsolete: true
Attachment #8975818 - Flags: review?(jgilbert)
This is the last missing piece to build Wayland enabled builds on Linux/Gtk.
Comment on attachment 8975816 [details]
Bug 1460255 - Implement GLContextProviderWayland,

https://reviewboard.mozilla.org/r/244010/#review253364
Attachment #8975816 - Flags: review?(lsalzman) → review+
Comment on attachment 8975817 [details]
Bug 1460255 - Use GLContextProviderWayland GL provider on Wayland enabled builds,

https://reviewboard.mozilla.org/r/244012/#review253366
Attachment #8975817 - Flags: review?(lsalzman) → review+
(In reply to Martin Stránský [:stransky] from comment #10)
> This is the last missing piece to build Wayland enabled builds on Linux/Gtk.

since it's reviewed, can it be pushed?
(In reply to David Heidelberg from comment #13)
> (In reply to Martin Stránský [:stransky] from comment #10)
> > This is the last missing piece to build Wayland enabled builds on Linux/Gtk.
> 
> since it's reviewed, can it be pushed?

Please wait until try finishes:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=3f75e43ecb659cbafbeb2332f46eda0085fea971
Duplicate of this bug: 1464994
Pushed by stransky@redhat.com:
https://hg.mozilla.org/integration/autoland/rev/446eb68ac1c3
Implement GLContextProviderWayland, r=lsalzman
https://hg.mozilla.org/integration/autoland/rev/229a64156f70
Use GLContextProviderWayland GL provider on Wayland enabled builds, r=lsalzman
https://hg.mozilla.org/mozilla-central/rev/446eb68ac1c3
https://hg.mozilla.org/mozilla-central/rev/229a64156f70
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.