diff options
author | Ryan Mitchell <rtmitchell@google.com> | 2020-11-15 21:41:24 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2020-11-15 21:41:24 +0000 |
commit | 8f2f4e14deb73eee88094a40091c1cbaf2952d26 (patch) | |
tree | cdab2fb44da8a1b01d172b8165ba044080389511 /libs | |
parent | ba6f39518816a8e0060bfa90b9b7cb0472613e7d (diff) | |
parent | 527ebbaa55f42e90a05f12f0154294d2e9d16903 (diff) |
Merge "Fix DominatorTree for locale and mcc/mnc config"
Diffstat (limited to 'libs')
-rw-r--r-- | libs/androidfw/ConfigDescription.cpp | 5 |
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); } |