2028973 - Make sure proper startup events occur, recommend adjustments around `InitTranslationsBrowserState`

Status: RESOLVED FIXED

(Firefox for Android :: Translations, task)

Description

[Olivia Hall [:olivia]]

Translations queries a bunch of information at startup.

InitTranslationsBrowserState should check if this is a sensible thing to do by checking the pref and setting the store, if the AIControl switch is on.

Comment 1

[Olivia Hall [:olivia]]

Attachment: Bug 2028973 - Do not initialize translations when the user has the feature off

Patch adds isTranslationsEnabled that uses TranslationsEnabledSettings.dataStore(context).isEnabled.first() to determine if the user wants translations enabled or not.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/7497097fd9bb
https://hg.mozilla.org/integration/autoland/rev/8a9f54364af7
Do not initialize translations when the user has the feature off r=matt-tighe,android-reviewers

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/8a9f54364af7