From 0e1c45fd8ba86e295c1b118fd3d4a1bc4b143dec Mon Sep 17 00:00:00 2001
From: Robert Greenwalt <>
Date: Mon, 13 Apr 2009 18:35:04 -0700
Subject: AI 145995: am: CL 145993 Fix Phone-oriented default-locale selection.
   When several locales for a given language are available we were always  
 selecting the first.   Original author: rgreenwalt   Merged from:
 //branches/cupcake/...

Automated import of CL 145995
---
 telephony/java/com/android/internal/telephony/PhoneBase.java | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'telephony')

diff --git a/telephony/java/com/android/internal/telephony/PhoneBase.java b/telephony/java/com/android/internal/telephony/PhoneBase.java
index 259de62..20c54fb 100644
--- a/telephony/java/com/android/internal/telephony/PhoneBase.java
+++ b/telephony/java/com/android/internal/telephony/PhoneBase.java
@@ -491,7 +491,7 @@ public abstract class PhoneBase implements Phone {
             return; // no match possible
         }
         language.toLowerCase();
-        if (null != country) {
+        if (null == country) {
             country = "";
         }
         country = country.toUpperCase();
@@ -505,10 +505,11 @@ public abstract class PhoneBase implements Phone {
                 for(int i = 0; i < N; i++) {
                     if (locales[i]!=null && locales[i].length() >= 2 &&
                             locales[i].substring(0,2).equals(language)) {
-                        if (locales[i].length() >= 5 &&
-                                locales[i].substring(3,5).equals(country)) {
-                            bestMatch = locales[i];
-                            break;
+                        if (locales[i].length() >= 5) {
+                            if (locales[i].substring(3,5).equals(country)) {
+                                bestMatch = locales[i];
+                                break;
+                            }
                         } else if (null == bestMatch) {
                             bestMatch = locales[i];
                         }
@@ -517,7 +518,6 @@ public abstract class PhoneBase implements Phone {
                 if (null != bestMatch) {
                     IActivityManager am = ActivityManagerNative.getDefault();
                     Configuration config = am.getConfiguration();
-
                     if (bestMatch.length() >= 5) {
                         config.locale = new Locale(bestMatch.substring(0,2),
                                                    bestMatch.substring(3,5));
-- 
cgit v1.1