1865010 - useDocumentFonts BrowserSetting value does not persist across restarts

Status: NEW

(GeckoView :: Extensions, defect, P3)

Description

[Gregory Pappas [:gregp]]

Steps to reproduce:

1. In a background script, run:

   browser.browserSettings.useDocumentFonts.set({ value: false });
   

2. Restart the browser
3. In a background script, run:

   let { value } = await browser.browserSettings.useDocumentFonts.get({});
   
   console.log(value);
   

Expected results:
false is logged to the console

Actual results:
true is logged to the console

I believe this is happening because:

• The underlying pref (browser.display.use_document_fonts) is controlled by GeckoRuntimeSettings
  https://searchfox.org/mozilla-central/rev/3d0b07f5a3c7b8d9955fb8e8acaaf582b1e33bb6/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoRuntimeSettings.java#512
• The user values of all prefs controlled by GeckoRuntimeSettings are reset on commit
  https://searchfox.org/mozilla-central/rev/3d0b07f5a3c7b8d9955fb8e8acaaf582b1e33bb6/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoRuntimeSettings.java#619

Comment 1

[Arturo Mejia [:amejia]]

This may be an issue related to GeckoView overridden values with its own default.