summaryrefslogtreecommitdiff
path: root/libs
diff options
context:
space:
mode:
authorRyan Mitchell <rtmitchell@google.com>2020-11-15 21:41:24 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2020-11-15 21:41:24 +0000
commit8f2f4e14deb73eee88094a40091c1cbaf2952d26 (patch)
treecdab2fb44da8a1b01d172b8165ba044080389511 /libs
parentba6f39518816a8e0060bfa90b9b7cb0472613e7d (diff)
parent527ebbaa55f42e90a05f12f0154294d2e9d16903 (diff)
Merge "Fix DominatorTree for locale and mcc/mnc config"
Diffstat (limited to 'libs')
-rw-r--r--libs/androidfw/ConfigDescription.cpp5
1 files changed, 4 insertions, 1 deletions
diff --git a/libs/androidfw/ConfigDescription.cpp b/libs/androidfw/ConfigDescription.cpp
index 1f3a89edb8af..19ead9583eb2 100644
--- a/libs/androidfw/ConfigDescription.cpp
+++ b/libs/androidfw/ConfigDescription.cpp
@@ -887,13 +887,16 @@ bool ConfigDescription::Dominates(const ConfigDescription& o) const {
}
// Locale de-duping is not-trivial, disable for now (b/62409213).
- if (diff(o) & CONFIG_LOCALE) {
+ // We must also disable de-duping for all configuration qualifiers with precedence higher than
+ // locale (b/171892595)
+ if (diff(o) & (CONFIG_LOCALE | CONFIG_MCC | CONFIG_MNC)) {
return false;
}
if (*this == DefaultConfig()) {
return true;
}
+
return MatchWithDensity(o) && !o.MatchWithDensity(*this) &&
!isMoreSpecificThan(o) && !o.HasHigherPrecedenceThan(*this);
}