Add possibility to choose EGL backend on Non-Maemo Linux/Qt/Gtk builds instead ofo GLX

RESOLVED FIXED in mozilla14

Status

()

Core
Graphics
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: romaxa, Assigned: romaxa)

Tracking

Trunk
mozilla14
ARM
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

5 years ago
Currently we are compiling for Linux target by default GLX provider.
And for maemo EGL provider.
I think it would be more flexible to have option which would allow to enable EGL backend for special configurations.
(Assignee)

Updated

5 years ago
Blocks: 735598
(Assignee)

Comment 1

5 years ago
Created attachment 605693 [details] [diff] [review]
Allow to use EGL provider for custom Linux targets
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #605693 - Flags: review?(mh+mozilla)
Attachment #605693 - Flags: review?(matt.woodrow)
Comment on attachment 605693 [details] [diff] [review]
Allow to use EGL provider for custom Linux targets

Review of attachment 605693 [details] [diff] [review]:
-----------------------------------------------------------------

::: configure.in
@@ +5186,5 @@
> +then
> +    MOZ_WEBGL_GLX=
> +    AC_SUBST(MOZ_FORCE_EGL_BACKEND)
> +    AC_DEFINE(MOZ_FORCE_EGL_BACKEND)
> +fi

Please make this more generic. I'd suggest --with-gl-provider=value. value would override the GL_PROVIDER value from gfx/gl/Makefile.in.
Attachment #605693 - Flags: review?(mh+mozilla) → review-
(Assignee)

Comment 3

5 years ago
Created attachment 605938 [details] [diff] [review]
Comments addresed
Attachment #605693 - Attachment is obsolete: true
Attachment #605693 - Flags: review?(matt.woodrow)
Attachment #605938 - Flags: review?(mh+mozilla)
Comment on attachment 605938 [details] [diff] [review]
Comments addresed

Review of attachment 605938 [details] [diff] [review]:
-----------------------------------------------------------------

I'd go further in GLContextProvider.h, removing now redundant things, and would also use this facility for windows and mac (only special case being EGL always being there on windows in addition to WGL). But I'd rather a gfx peer to give their opinion.
Attachment #605938 - Flags: review?(mh+mozilla) → feedback?(jmuizelaar)
(Assignee)

Comment 5

5 years ago
Created attachment 608549 [details] [diff] [review]
Custom GL provider config option
Attachment #605938 - Attachment is obsolete: true
Attachment #605938 - Flags: feedback?(jmuizelaar)
Attachment #608549 - Flags: review?(jgilbert)
Comment on attachment 608549 [details] [diff] [review]
Custom GL provider config option

Review of attachment 608549 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine to me, but I don't know enough about the configure stuff.

Maybe bjacob knows better?
Attachment #608549 - Flags: review?(jgilbert)
Attachment #608549 - Flags: review?(bjacob)
Attachment #608549 - Flags: feedback+
(Assignee)

Comment 7

5 years ago
Mike already checked configure stuff
(In reply to Jeff Gilbert [:jgilbert] from comment #6)
> Comment on attachment 608549 [details] [diff] [review]
> Custom GL provider config option
> 
> Review of attachment 608549 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks fine to me, but I don't know enough about the configure stuff.
> 
> Maybe bjacob knows better?

Just in case, did you see comment 4?
Comment on attachment 608549 [details] [diff] [review]
Custom GL provider config option

Alright, sounds good then.
Attachment #608549 - Flags: review?(bjacob)
Attachment #608549 - Flags: review+
Attachment #608549 - Flags: feedback+
Please can you set the target milestone when landing on inbound, along the lines of http://blog.bonardo.net/2012/03/23/how-you-can-help-mozilla-inbound-sheriffs-when-pushing :-)

https://hg.mozilla.org/mozilla-central/rev/6bbe864b5162
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.