AboutThisSite: Enable PFINGS by default on Android
Bug: b/271425975
Change-Id: I7172711a6976d5e6b15cad1406dbe4d6b5de7216
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4341748
Commit-Queue: Christian Dullweber <[email protected]>
Reviewed-by: Filipa Senra <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1118016}
diff --git a/components/page_info/core/features.cc b/components/page_info/core/features.cc
index 7438ddf..c107a39 100644
--- a/components/page_info/core/features.cc
+++ b/components/page_info/core/features.cc
@@ -4,6 +4,7 @@
#include "components/page_info/core/features.h"
+#include "base/containers/contains.h"
#include "base/feature_list.h"
#include "base/metrics/field_trial_params.h"
#include "build/build_config.h"
@@ -20,10 +21,18 @@
base::FEATURE_DISABLED_BY_DEFAULT);
#endif
+const char* pfings[]{"pt", "fr", "it", "nl", "de", "es"};
+
extern bool IsAboutThisSiteFeatureEnabled(const std::string& locale) {
- if (l10n_util::GetLanguage(locale) == "en") {
+ std::string lang = l10n_util::GetLanguage(locale);
+ if (lang == "en") {
return base::FeatureList::IsEnabled(kPageInfoAboutThisSiteEn);
} else {
+#if BUILDFLAG(IS_ANDROID)
+ if (base::Contains(pfings, lang)) {
+ return base::FeatureList::IsEnabled(kPageInfoAboutThisSiteEn);
+ }
+#endif
return base::FeatureList::IsEnabled(kPageInfoAboutThisSiteNonEn);
}
}
