Bug 1491209 - Do not inject last fallback locale into requested local…

…es. r=jfkthame

Differential Revision: https://phabricator.services.mozilla.com/D5839

--HG--
extra : moz-landing-system : lando

intl/locale/LocaleService.cpp

@@ -122,15 +122,6 @@ ReadRequestedLocales(nsTArray<nsCString>& aRetVal)
     aRetVal.AppendElement(defaultLocale);
   }
 
-  // Last fallback locale is a locale for the requested locale chain.
-  // In the future we'll want to make the fallback chain differ per-locale.
-  //
-  // Notice: This is not the same as DefaultLocale,
-  // which follows the default locale the build is in.
-  LocaleService::GetInstance()->GetLastFallbackLocale(str);
-  if (!aRetVal.Contains(str)) {
-    aRetVal.AppendElement(str);
-  }
   return true;
 }
 

intl/locale/tests/unit/test_localeService.js

@@ -173,8 +173,7 @@ add_test(function test_getRequestedLocales_sanitize() {
   Assert.equal(locales[1], "pl");
   Assert.equal(locales[2], "de-AT");
   Assert.equal(locales[3], "und");
-  Assert.equal(locales[4], localeService.lastFallbackLocale);
-  Assert.equal(locales.length, 5);
+  Assert.equal(locales.length, 4);
 
   Services.prefs.clearUserPref(PREF_REQUESTED_LOCALES);