Bug 1960383 - Remove vestigial logic related to browser.translations.panelShown r=nordzilla,translations-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D245636
This commit is contained in:
@@ -2641,9 +2641,22 @@ export var Policies = {
|
|||||||
param.FeatureRecommendations,
|
param.FeatureRecommendations,
|
||||||
param.Locked
|
param.Locked
|
||||||
);
|
);
|
||||||
|
|
||||||
|
// We use the mostRecentTargetLanguages pref to control the
|
||||||
|
// translations panel intro. Setting a language value simulates a
|
||||||
|
// first translation, which skips the intro panel for users with
|
||||||
|
// FeatureRecommendations disabled.
|
||||||
|
const topWebPreferredLanguage = Services.prefs
|
||||||
|
.getComplexValue("intl.accept_languages", Ci.nsIPrefLocalizedString)
|
||||||
|
.data.split(/\s*,\s*/g)[0];
|
||||||
|
|
||||||
|
const preferredLanguage = topWebPreferredLanguage.length
|
||||||
|
? topWebPreferredLanguage
|
||||||
|
: Services.locale.appLocaleAsBCP47;
|
||||||
|
|
||||||
PoliciesUtils.setDefaultPref(
|
PoliciesUtils.setDefaultPref(
|
||||||
"browser.translations.panelShown",
|
"browser.translations.mostRecentTargetLanguages",
|
||||||
!param.FeatureRecommendations,
|
param.FeatureRecommendations ? "" : preferredLanguage,
|
||||||
param.Locked
|
param.Locked
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -11,7 +11,6 @@ add_task(async function test_translations_telemetry_auto_translate() {
|
|||||||
const { cleanup, resolveDownloads, runInPage } = await loadTestPage({
|
const { cleanup, resolveDownloads, runInPage } = await loadTestPage({
|
||||||
page: SPANISH_PAGE_URL,
|
page: SPANISH_PAGE_URL,
|
||||||
languagePairs: LANGUAGE_PAIRS,
|
languagePairs: LANGUAGE_PAIRS,
|
||||||
prefs: [["browser.translations.panelShown", false]],
|
|
||||||
});
|
});
|
||||||
|
|
||||||
await FullPageTranslationsTestUtils.assertTranslationsButton(
|
await FullPageTranslationsTestUtils.assertTranslationsButton(
|
||||||
|
|||||||
@@ -10,7 +10,6 @@ add_task(async function test_translations_telemetry_basics() {
|
|||||||
const { cleanup } = await loadTestPage({
|
const { cleanup } = await loadTestPage({
|
||||||
page: SPANISH_PAGE_URL,
|
page: SPANISH_PAGE_URL,
|
||||||
languagePairs: LANGUAGE_PAIRS,
|
languagePairs: LANGUAGE_PAIRS,
|
||||||
prefs: [["browser.translations.panelShown", false]],
|
|
||||||
});
|
});
|
||||||
|
|
||||||
await TestTranslationsTelemetry.assertEvent(Glean.translationsPanel.open, {
|
await TestTranslationsTelemetry.assertEvent(Glean.translationsPanel.open, {
|
||||||
|
|||||||
@@ -14,7 +14,6 @@ add_task(async function test_translations_telemetry_unsupported_lang() {
|
|||||||
{ fromLang: "fr", toLang: "en" },
|
{ fromLang: "fr", toLang: "en" },
|
||||||
{ fromLang: "en", toLang: "fr" },
|
{ fromLang: "en", toLang: "fr" },
|
||||||
],
|
],
|
||||||
prefs: [["browser.translations.panelShown", false]],
|
|
||||||
});
|
});
|
||||||
|
|
||||||
await FullPageTranslationsTestUtils.openPanel({
|
await FullPageTranslationsTestUtils.openPanel({
|
||||||
|
|||||||
@@ -425,17 +425,6 @@ export class TranslationsParent extends JSWindowActorParent {
|
|||||||
*/
|
*/
|
||||||
resolveEngine = null;
|
resolveEngine = null;
|
||||||
|
|
||||||
/**
|
|
||||||
* The cached URI spec where the panel was first ever shown, as determined by the
|
|
||||||
* browser.translations.panelShown pref.
|
|
||||||
*
|
|
||||||
* Holding on to this URI value allows us to show the introductory message in the panel
|
|
||||||
* when the panel opens, as long as the active panel is open on that particular URI.
|
|
||||||
*
|
|
||||||
* @type {string | null}
|
|
||||||
*/
|
|
||||||
firstShowUriSpec = null;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* The TranslationsEngineParent instance which requests from this
|
* The TranslationsEngineParent instance which requests from this
|
||||||
* TranslationsParent are being handled by.
|
* TranslationsParent are being handled by.
|
||||||
|
|||||||
@@ -1050,7 +1050,6 @@ async function loadTestPage({
|
|||||||
// Enabled by default.
|
// Enabled by default.
|
||||||
["browser.translations.enable", true],
|
["browser.translations.enable", true],
|
||||||
["browser.translations.logLevel", "All"],
|
["browser.translations.logLevel", "All"],
|
||||||
["browser.translations.panelShown", true],
|
|
||||||
["browser.translations.automaticallyPopup", true],
|
["browser.translations.automaticallyPopup", true],
|
||||||
["browser.translations.alwaysTranslateLanguages", ""],
|
["browser.translations.alwaysTranslateLanguages", ""],
|
||||||
["browser.translations.neverTranslateLanguages", ""],
|
["browser.translations.neverTranslateLanguages", ""],
|
||||||
|
|||||||
Reference in New Issue
Block a user