Remove Norwegian Nynorsk from UI supported languages.

Note that this still lets you selecting the language and ordering for accept-language requests, just that the option of using the language as "System text" is disabled.

Currently we offer Norwegian Nynorsk as an UI language.
The removed code tries to map it to Norwegian Bokmål, but these two languages are similar, but not the same (https://www.skapago.eu/en/bokmal-nynorsk/).
Furthermore, since we don't actually map it (https://source.chromium.org/chromium/chromium/src/+/master:ui/base/l10n/l10n_util.cc;drc=4d93574ee586e569d9412f837c74824eb79ba02e?originalUrl=https:%2F%2Fcs.chromium.org%2F)
the language actually fall-backs to english when selected.
Since Norwegian Nynorsk is actually not a supported language, it is better to not offer it as a system language.

TEST=manual (ChromeOS on linux)


Bug: 1044756
Change-Id: Ice228606313c603cd1214703b024b8f370c5ccb1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2024128
Commit-Queue: David Vallet <dvallet@chromium.org>
Reviewed-by: David Vallet <dvallet@chromium.org>
Reviewed-by: Jia Meng <jiameng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#735754}

components/language/core/common/locale_util.cc

@@ -25,16 +25,16 @@ struct LocaleUIFallbackPair {
 
 // This list MUST be sorted by the first element in the pair, because we perform
 // binary search on it.
-// TODO(claudiomagni): Investigate Norvegian language. There are 2 codes ("nn",
-// "no") that fallback to "nb", but the base language should be "nn".
+// Note that no (Norwegian) is an alias, and should fallback to Norwegian
+// Bokmål (nb)
 const LocaleUIFallbackPair kLocaleUIFallbackTable[] = {
     {"en", "en-US"},     {"en-AU", "en-GB"},  {"en-CA", "en-GB"},
     {"en-IN", "en-GB"},  {"en-NZ", "en-GB"},  {"en-ZA", "en-GB"},
     {"es-AR", "es-419"}, {"es-CL", "es-419"}, {"es-CO", "es-419"},
     {"es-CR", "es-419"}, {"es-HN", "es-419"}, {"es-MX", "es-419"},
     {"es-PE", "es-419"}, {"es-US", "es-419"}, {"es-UY", "es-419"},
-    {"es-VE", "es-419"}, {"it-CH", "it"},     {"nn", "nb"},
-    {"no", "nb"},        {"pt", "pt-PT"}};
+    {"es-VE", "es-419"}, {"it-CH", "it"},     {"no", "nb"},
+    {"pt", "pt-PT"}};
 
 bool LocaleCompare(const LocaleUIFallbackPair& p1, const std::string& p2) {
   return p1.chosen_locale < p2;

components/language/core/common/locale_util_unittest.cc

@@ -212,11 +212,6 @@ TEST_F(LocaleUtilTest, ConvertToActualUILocale) {
   EXPECT_TRUE(is_ui);
   EXPECT_EQ("it", locale);
 
-  locale = "nn";
-  is_ui = language::ConvertToActualUILocale(&locale);
-  EXPECT_TRUE(is_ui);
-  EXPECT_EQ("nb", locale);
-
   locale = "no";
   is_ui = language::ConvertToActualUILocale(&locale);
   EXPECT_TRUE(is_ui);