Navi: Don't show email module outside en-US.
This is because we don't have a good list of email providers to
recommend that will be relevant based on locale.
Bug: 924274
Change-Id: I432c1a81e0c0272673edace4ac82781dbb216c45
Reviewed-on: https://chromium-review.googlesource.com/c/1437554
Commit-Queue: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org>
Auto-Submit: Hector Carmona <hcarmona@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#626321}(cherry picked from commit ae899993e3929690eb0e69ed028d454e6b9ae944)
Reviewed-on: https://chromium-review.googlesource.com/c/1449006
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/branch-heads/3683@{#99}
Cr-Branched-From: e51029943e0a38dd794b73caaf6373d5496ae783-refs/heads/master@{#625896}
diff --git a/chrome/browser/resources/welcome/onboarding_welcome/welcome_app.js b/chrome/browser/resources/welcome/onboarding_welcome/welcome_app.js
index 43d7827..c988dbd 100644
--- a/chrome/browser/resources/welcome/onboarding_welcome/welcome_app.js
+++ b/chrome/browser/resources/welcome/onboarding_welcome/welcome_app.js
@@ -131,9 +131,20 @@
])
.then(args => {
const canSetDefault = args[0];
- if (!canSetDefault) {
- modules = modules.filter(module => module != 'nux-set-as-default');
- }
+
+ modules = modules.filter(module => {
+ if (module == 'nux-set-as-default') {
+ return canSetDefault;
+ }
+
+ if (module == 'nux-email') {
+ // Show email module in en-US only until email recommendations
+ // for other locales is figured out.
+ return navigator.language == 'en-US';
+ }
+
+ return true;
+ });
const indicatorElementCount = modules.reduce((count, module) => {
return count += MODULES_NEEDING_INDICATOR.has(module) ? 1 : 0;