1900106 - libxul C++ constructors run before setupGeckoEnvironment() can set up the enviroment

Status: NEW

(GeckoView :: General, defect, P3)

Description

[Jeff Muizelaar [:jrmuizel]]

GeckoView has the ability to setup environment variables via a config file. Currently we're loading libxul via glean (bug 1892234) instead of explicitly in GeckoLoader. This causes us to run C++ constructors before we setup the environment variables that we want for Gecko.

We set up the appropriate environment variables for gecko here:
https://searchfox.org/mozilla-central/rev/fc76676f61ee37b4c5420649cad6677164a29405/mobile/android/geckoview/src/main/java/org/mozilla/gecko/mozglue/GeckoLoader.java#116

Then explicitly load libxul here: https://searchfox.org/mozilla-central/rev/fc76676f61ee37b4c5420649cad6677164a29405/mobile/android/geckoview/src/main/java/org/mozilla/gecko/mozglue/GeckoLoader.java#179

I noticed this because it was happening for DefaultJitOptions

It's also worth noting that this causes inconsistent results for the environment variables in the parent process and the content process.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:owlish, could you have a look please?

For more information, please visit BugBot documentation.