Closed Bug 1536872 Opened 5 years ago Closed 5 years ago

about:config doesn't save pref changes in FFTV+GV

Categories

(GeckoView :: General, defect, P1)

All
Android
defect

Tracking

(firefox66 wontfix, firefox67 wontfix, firefox68 fixed)

RESOLVED WORKSFORME
Tracking Status
firefox66 --- wontfix
firefox67 --- wontfix
firefox68 --- fixed

People

(Reporter: cpeterson, Unassigned)

References

()

Details

(Whiteboard: [gvtv:p1][geckoview:fenix:m4])

Simon says about:config pref changes aren't saved after he restarts FFTV+GV:

https://github.com/mozilla-mobile/firefox-tv/issues/1980

STR:

  1. go to about:config
  2. set media.media-capabilities.enabled to false
  3. restart app

Expected behavior:
media.media-capabilities.enabled is still set to false

Actual behavior:
media.media-capabilities.enabled is back to true (default value for 67) after FFTV restart. Pref changes are remembered if you close and reopen about:config before quitting FFTV.

James thinks this is the process switching bug 1527778 because about:config runs in the parent process.

Depends on: 1527778
Whiteboard: [gvtv:p1] → [gvtv:p1][geckoview:fenix:m4]

67=wontfix. Fenix MVP will use GeckoView 68, so we don't need to uplift this fix to 67 Beta.

Simon, is this about:config bug still reproducible with a recent snapshot of GV 68? cpearce says that he regularly changes pref values in about:config in FFTV and the new values are correctly saved across sessions.

Flags: needinfo?(schae)

(In reply to Chris Peterson [:cpeterson] from comment #3)

Simon, is this about:config bug still reproducible with a recent snapshot of GV 68? cpearce says that he regularly changes pref values in about:config in FFTV and the new values are correctly saved across sessions.

No longer reproducible with nightly 68.0.20190414095735

Flags: needinfo?(schae)

(In reply to Simon Chae from comment #4)

No longer reproducible with nightly 68.0.20190414095735

Awesome! Thanks for checking.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.