From 1e9d497e2b0a3d00e83372d536b79c9ec0646b9d Mon Sep 17 00:00:00 2001
From: Daisuke Miyakawa <dmiyakawa@google.com>
Date: Mon, 15 Nov 2010 12:41:06 -0800
Subject: [PATCH 1/4] Show option in DateTimeSettings.

Change-Id: Ia6bd6cc22ec154b83879c488274bd2b0cfe238d2
---
 src/com/android/settings/ZonePicker.java | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/com/android/settings/ZonePicker.java b/src/com/android/settings/ZonePicker.java
index f0c61edc819..3de4090457e 100644
--- a/src/com/android/settings/ZonePicker.java
+++ b/src/com/android/settings/ZonePicker.java
@@ -108,6 +108,7 @@ public class ZonePicker extends ListFragment {
 
         // If current timezone is in this list, move focus to it
         setSelection(mDefault);
+        setHasOptionsMenu(true);
     }
 
     @Override

From 2571f0dcb1cc96b7220687aff6fcddc623b57c3c Mon Sep 17 00:00:00 2001
From: Daisuke Miyakawa <dmiyakawa@google.com>
Date: Fri, 29 Oct 2010 15:01:56 -0700
Subject: [PATCH 2/4] Refactor DateTimeSettingsSetup.

- Use Popup instead of bare fragment.
- Expose Adapters in ZonePicker.java so that it can be used
  outside the fragment.
- Fix layout

Bug: 3175603
Change-Id: I2726fde4fa1a9aea1ecb29b6aa2d23dbc54232b9
---
 .../date_time_settings_setupwizard.xml        | 224 +++++++++---------
 .../wifi_settings_for_setup_wizard_xl.xml     |  13 +-
 .../settings/DateTimeSettingsSetupWizard.java |  61 +++--
 src/com/android/settings/ZonePicker.java      | 147 +++++++-----
 4 files changed, 253 insertions(+), 192 deletions(-)

diff --git a/res/layout-xlarge/date_time_settings_setupwizard.xml b/res/layout-xlarge/date_time_settings_setupwizard.xml
index 254428f936b..d9297d01e2c 100644
--- a/res/layout-xlarge/date_time_settings_setupwizard.xml
+++ b/res/layout-xlarge/date_time_settings_setupwizard.xml
@@ -14,66 +14,61 @@
      limitations under the License.
   -->
 <!-- TODO: too many LinearLayout. -->
-<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-              android:orientation="vertical"
-              android:layout_width="fill_parent"
-              android:layout_height="fill_parent"
-              android:keepScreenOn="true"
-              android:paddingTop="70dip"
-              android:paddingBottom="100dip"
-              android:paddingLeft="60dip"
-              android:paddingRight="60dip">
+<RelativeLayout
+    xmlns:android="http://schemas.android.com/apk/res/android"
+    android:orientation="vertical"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    android:keepScreenOn="true"
+    android:paddingTop="60dip"
+    android:paddingLeft="100dip"
+    android:paddingRight="100dip"
+    android:paddingBottom="40dip">
 
     <!-- Title: Set date & time-->
-    <TextView android:id="@+id/title"
-              android:layout_width="fill_parent"
-              android:layout_height="wrap_content"
-              android:layout_weight="0"
-              android:gravity="center"
-              android:layout_marginBottom="30dip"
-              android:layout_alignParentTop="true"
-              android:textSize="64dip"
-              android:textColor="#FF99cc00"
-              android:text="@string/date_and_time_settings_title"/>
+    <TextView
+        android:id="@+id/title"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_alignParentLeft="true"
+        android:layout_alignParentTop="true"
+        android:textSize="40dip"
+        android:textColor="#FF99cc00"
+        android:text="@string/date_and_time_settings_title"/>
 
-    <LinearLayout android:id="@+id/main"
-                  android:orientation="horizontal"
-                  android:layout_width="fill_parent"
-                  android:layout_height="fill_parent"
-                  android:layout_weight="1">
-        <!-- Left side: time zone setting -->
-        <RelativeLayout android:layout_width="0px"
-                        android:layout_weight="1"
-                        android:layout_height="fill_parent"
-                        android:layout_marginRight="50dip">
-            <LinearLayout android:id="@+id/timezone"
-                          android:orientation="vertical"
-                          android:layout_width="fill_parent"
-                          android:layout_height="wrap_content"
-                          android:layout_weight="1"
-                          android:layout_alignParentTop="true">
+    <!-- Divider -->
+    <View
+        android:id="@+id/top_divider"
+        android:layout_width="match_parent"
+        android:layout_height="1px"
+        android:layout_below="@id/title"
+        android:background="@color/divider_color"
+        android:layout_marginBottom="5dip" />
 
-                <!--
-                <CheckBox android:id="@+id/time_zone_auto"
-                          android:layout_width="wrap_content"
-                          android:layout_height="wrap_content"
-                          android:gravity="right|center_horizontal"
-                          android:layout_marginBottom="5dip"
-                          android:textSize="32dip"
-                          android:text="@string/time_zone_auto_stub"/> -->
+    <LinearLayout
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_below="@id/top_divider"
+        android:layout_alignParentLeft="true"
+        android:layout_marginTop="60dip"
+        android:orientation="horizontal">
 
-                <!-- text should manually be set. -->
-                <Button android:id="@+id/current_time_zone"
-                        android:layout_width="fill_parent"
-                        android:layout_height="wrap_content"
-                        android:textSize="24dip"
-                        android:layout_alignParentTop="true" />
-                <LinearLayout android:id="@+id/zone_picker"
-                              android:orientation="vertical"
+        <!-- left: timezone -->
+        <LinearLayout
+            android:layout_width="0px"
+            android:layout_weight=".48"
+            android:layout_height="wrap_content"
+            android:orientation="vertical">
+            <!-- text should manually be set. -->
+            <Button
+                 android:id="@+id/time_zone_button"
+                 android:layout_width="400dip"
+                 android:layout_height="60dip"
+                 android:textSize="24dip" />
+            <!-- <LinearLayout android:id="@+id/zone_picker"
+                               android:orientation="vertical"
                               android:layout_width="fill_parent"
                               android:layout_height="350dip"
-                              android:paddingLeft="10dip"
-                              android:paddingBottom="20dip"
                               android:gravity="center"
                               android:visibility="gone"
                               android:clickable="true">
@@ -81,68 +76,75 @@
                               class="com.android.settings.ZonePicker"
                               android:layout_width="fill_parent"
                               android:layout_height="wrap_content" />
-                </LinearLayout>
-            </LinearLayout>
-            <Button android:id="@+id/skip_button"
-                    android:layout_width="300dip"
-                    android:layout_height="wrap_content"
-                    android:layout_weight="0"
-                    android:layout_alignParentBottom="true"
-                    android:layout_centerHorizontal="true"
-                    android:textSize="32dip"
-                    android:text="@string/skip_label"/>
-        </RelativeLayout>
+            </LinearLayout> -->
+        </LinearLayout>
 
-        <!-- Right side: date & time setting -->
-        <RelativeLayout android:layout_width="0px"
-                        android:layout_weight="1"
-                        android:layout_height="fill_parent"
-                        android:layout_marginLeft="50dip">
-            <LinearLayout android:id="@+id/datetime"
-                          android:orientation="vertical"
-                          android:layout_width="fill_parent"
-                          android:layout_height="wrap_content"
-                          android:layout_weight="1"
-                          android:layout_alignParentTop="true">
+        <!-- spacer on center -->
+        <View
+            android:layout_width="0px"
+            android:layout_weight=".04"
+            android:layout_height="0dip"
+            android:visibility="invisible" />
 
-                <CheckBox android:id="@+id/date_time_auto"
-                          android:layout_width="wrap_content"
-                          android:layout_height="wrap_content"
-                          android:gravity="right|center_horizontal"
-                          android:textSize="32dip"
-                          android:text="@string/date_time_auto" />
+        <!-- right: DateTime -->
+        <RelativeLayout
+            android:layout_width="0px"
+            android:layout_weight=".48"
+            android:layout_height="wrap_content">
+
+            <CheckBox
+                android:id="@+id/date_time_auto"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:layout_alignParentTop="true"
+                android:gravity="left"
+                android:textSize="22dip"
+                android:text="@string/date_time_auto" />
+
+            <!-- Divider -->
+            <View
+                android:id="@+id/datetime_divider"
+                android:layout_width="match_parent"
+                android:layout_height="1px"
+                android:layout_below="@id/date_time_auto"
+                android:background="@color/divider_color"
+                android:layout_marginTop="15dip"
+                android:layout_marginBottom="80dip" />
+
+            <TimePicker
+                android:id="@+id/time_picker"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:layout_alignParentLeft="true"
+                android:layout_below="@id/datetime_divider" />
+
+            <DatePicker
+                android:id="@+id/date_picker"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:layout_alignParentRight="true"
+                android:layout_below="@id/datetime_divider" />
 
-                <LinearLayout android:orientation="horizontal"
-                              android:layout_width="fill_parent"
-                              android:layout_height="wrap_content">
-                    <TimePicker android:id="@+id/time_picker"
-                                android:layout_width="0px"
-                                android:layout_weight=".5"
-                                android:layout_height="wrap_content"
-                                android:visibility="visible"/>
-                    <DatePicker android:id="@+id/date_picker"
-                                android:layout_width="0px"
-                                android:layout_weight=".5"
-                                android:layout_height="wrap_content"/>
-                </LinearLayout>
-            </LinearLayout>
-            <Button android:id="@+id/next_button"
-                    android:layout_width="300dip"
-                    android:layout_height="wrap_content"
-                    android:layout_weight="0"
-                    android:layout_alignParentBottom="true"
-                    android:layout_centerHorizontal="true"
-                    android:textSize="32dip"
-                    android:text="@string/next_label" />
         </RelativeLayout>
     </LinearLayout>
 
-    <LinearLayout android:id="@+id/bottom"
-                  android:orientation="horizontal"
-                  android:layout_width="fill_parent"
-                  android:gravity="center"
-                  android:layout_height="wrap_content"
-                  android:layout_weight="0">
+    <Button
+        android:id="@+id/skip_button"
+        android:layout_width="250dip"
+        android:layout_height="80dip"
+        android:layout_alignParentBottom="true"
+        android:layout_toLeftOf="@+id/next_button"
+        android:layout_marginRight="20dip"
+        android:textSize="24dip"
+        android:text="@string/skip_label"/>
 
-    </LinearLayout>
-</LinearLayout>
+    <Button
+        android:id="@+id/next_button"
+        android:layout_width="250dip"
+        android:layout_height="80dip"
+        android:layout_alignParentBottom="true"
+        android:layout_alignParentRight="true"
+        android:textSize="24dip"
+        android:text="@string/next_label" />
+
+</RelativeLayout>
diff --git a/res/layout-xlarge/wifi_settings_for_setup_wizard_xl.xml b/res/layout-xlarge/wifi_settings_for_setup_wizard_xl.xml
index 1244f2aa7ba..97b8d2d8125 100644
--- a/res/layout-xlarge/wifi_settings_for_setup_wizard_xl.xml
+++ b/res/layout-xlarge/wifi_settings_for_setup_wizard_xl.xml
@@ -30,18 +30,17 @@
         android:layout_height="wrap_content"
         android:layout_alignParentLeft="true"
         android:layout_alignParentTop="true"
-        android:gravity="left"
         android:textSize="40dip"
         android:textColor="#FF99cc00"
         android:text="@string/wifi_setup_title"/>
 
     <ProgressBar
-         android:id="@+id/scanning_progress_bar"
-         android:layout_width="100dip"
-         android:layout_height="wrap_content"
-         android:layout_above="@+id/scanning_progress_text"
-         android:layout_alignParentRight="true"
-         style="?android:attr/progressBarStyleHorizontal" />
+        android:id="@+id/scanning_progress_bar"
+        android:layout_width="100dip"
+        android:layout_height="wrap_content"
+        android:layout_above="@+id/scanning_progress_text"
+        android:layout_alignParentRight="true"
+        style="?android:attr/progressBarStyleHorizontal" />
 
     <TextView
         android:id="@+id/scanning_progress_text"
diff --git a/src/com/android/settings/DateTimeSettingsSetupWizard.java b/src/com/android/settings/DateTimeSettingsSetupWizard.java
index 92bcf7d6c41..ad529f9469c 100644
--- a/src/com/android/settings/DateTimeSettingsSetupWizard.java
+++ b/src/com/android/settings/DateTimeSettingsSetupWizard.java
@@ -16,30 +16,34 @@
 
 package com.android.settings;
 
-import com.android.settings.ZonePicker.ZoneSelectionListener;
-
 import android.app.Activity;
+import android.app.AlarmManager;
 import android.content.Context;
 import android.content.Intent;
 import android.content.res.Configuration;
 import android.os.Bundle;
 import android.provider.Settings;
 import android.provider.Settings.SettingNotFoundException;
+import android.util.Log;
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.Window;
 import android.view.inputmethod.InputMethodManager;
+import android.widget.AdapterView;
+import android.widget.AdapterView.OnItemClickListener;
 import android.widget.Button;
 import android.widget.CompoundButton;
 import android.widget.CompoundButton.OnCheckedChangeListener;
 import android.widget.DatePicker;
+import android.widget.ListPopupWindow;
+import android.widget.SimpleAdapter;
 import android.widget.TimePicker;
 
 import java.util.Calendar;
 import java.util.TimeZone;
 
 public class DateTimeSettingsSetupWizard extends Activity
-        implements OnClickListener, ZoneSelectionListener, OnCheckedChangeListener{
+        implements OnClickListener, OnItemClickListener, OnCheckedChangeListener{
     private static final String TAG = DateTimeSettingsSetupWizard.class.getSimpleName();
 
     // force the first status of auto datetime flag.
@@ -51,7 +55,12 @@ public class DateTimeSettingsSetupWizard extends Activity
     /* Available only in XL */
     private CompoundButton mAutoDateTimeButton;
     // private CompoundButton mAutoTimeZoneButton;
-    private Button mTimeZone;
+
+    private Button mTimeZoneButton;
+    private ListPopupWindow mTimeZonePopup;
+    private SimpleAdapter mTimeZoneAdapter;
+    private TimeZone mSelectedTimeZone;
+
     private TimePicker mTimePicker;
     private DatePicker mDatePicker;
     private InputMethodManager mInputMethodManager;
@@ -83,10 +92,12 @@ public class DateTimeSettingsSetupWizard extends Activity
                 R.string.zone_auto_summaryOff);*/
 
         final TimeZone tz = TimeZone.getDefault();
-        mTimeZone = (Button)findViewById(R.id.current_time_zone);
-        mTimeZone.setText(DateTimeSettings.getTimeZoneText(tz));
-        mTimeZone.setOnClickListener(this);
-        // mTimeZone.setEnabled(!autoTimeZoneEnabled);
+        mSelectedTimeZone = tz;
+        mTimeZoneButton = (Button)findViewById(R.id.time_zone_button);
+        mTimeZoneButton.setText(tz.getDisplayName());
+        // mTimeZoneButton.setText(DateTimeSettings.getTimeZoneText(tz));
+        mTimeZoneButton.setOnClickListener(this);
+        mTimeZoneAdapter = ZonePicker.constructTimezoneAdapter(this, false);
 
         final boolean autoDateTimeEnabled;
         final Intent intent = getIntent();
@@ -109,9 +120,6 @@ public class DateTimeSettingsSetupWizard extends Activity
 
         mInputMethodManager = (InputMethodManager)getSystemService(Context.INPUT_METHOD_SERVICE);
 
-        ((ZonePicker)getFragmentManager().findFragmentById(R.id.zone_picker_fragment))
-                .setZoneSelectionListener(this);
-
         ((Button)findViewById(R.id.next_button)).setOnClickListener(this);
         ((Button)findViewById(R.id.skip_button)).setOnClickListener(this);
     }
@@ -119,9 +127,14 @@ public class DateTimeSettingsSetupWizard extends Activity
     @Override
     public void onClick(View view) {
         switch (view.getId()) {
-        case R.id.current_time_zone: {
-            findViewById(R.id.current_time_zone).setVisibility(View.GONE);
-            findViewById(R.id.zone_picker).setVisibility(View.VISIBLE);
+        case R.id.time_zone_button: {
+            mTimeZonePopup = new ListPopupWindow(this, null);
+            mTimeZonePopup.setWidth(mTimeZoneButton.getWidth());
+            mTimeZonePopup.setAnchorView(mTimeZoneButton);
+            mTimeZonePopup.setAdapter(mTimeZoneAdapter);
+            mTimeZonePopup.setOnItemClickListener(this);
+            mTimeZonePopup.setModal(true);
+            mTimeZonePopup.show();
             break;
         }
         case R.id.next_button: {
@@ -130,6 +143,15 @@ public class DateTimeSettingsSetupWizard extends Activity
                         mAutoTimeZoneButton.isChecked() ? 1 : 0); */
                 Settings.System.putInt(getContentResolver(), Settings.System.AUTO_TIME,
                         mAutoDateTimeButton.isChecked() ? 1 : 0);
+
+                final TimeZone systemTimeZone = TimeZone.getDefault();
+                if (!systemTimeZone.equals(mSelectedTimeZone)) {
+                    Log.i(TAG, "Another TimeZone is selected by a user. Changing system TimeZone.");
+                    final AlarmManager alarm = (AlarmManager)
+                            getSystemService(Context.ALARM_SERVICE);
+                    alarm.setTimeZone(mSelectedTimeZone.getID());
+                }
+
                 // Note: in non-XL, Date & Time is stored by DatePickerDialog/TimePickerDialog,
                 // so we don't need to save those values there, while in XL, we need to as
                 // we don't use those Dialogs.
@@ -182,15 +204,18 @@ public class DateTimeSettingsSetupWizard extends Activity
     }
 
     @Override
-    public void onZoneSelected(TimeZone tz) {
-        findViewById(R.id.current_time_zone).setVisibility(View.VISIBLE);
-        findViewById(R.id.zone_picker).setVisibility(View.GONE);
+    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
+        final TimeZone tz = ZonePicker.obtainTimeZoneFromItem(parent.getItemAtPosition(position));
+        mSelectedTimeZone = tz;
+
         final Calendar now = Calendar.getInstance(tz);
-        mTimeZone.setText(DateTimeSettings.getTimeZoneText(tz));
+        mTimeZoneButton.setText(tz.getDisplayName());
+        // mTimeZoneButton.setText(DateTimeSettings.getTimeZoneText(tz));
         mDatePicker.updateDate(now.get(Calendar.YEAR), now.get(Calendar.MONTH),
                 now.get(Calendar.DAY_OF_MONTH));
         mTimePicker.setCurrentHour(now.get(Calendar.HOUR));
         mTimePicker.setCurrentMinute(now.get(Calendar.MINUTE));
+        mTimeZonePopup.dismiss();
     }
 
     private boolean isAutoDateTimeEnabled() {
diff --git a/src/com/android/settings/ZonePicker.java b/src/com/android/settings/ZonePicker.java
index 3de4090457e..83d46cb35d6 100644
--- a/src/com/android/settings/ZonePicker.java
+++ b/src/com/android/settings/ZonePicker.java
@@ -55,10 +55,10 @@ public class ZonePicker extends ListFragment {
         public void onZoneSelected(TimeZone tz);
     }
 
-    private static final String KEY_ID = "id";
-    private static final String KEY_DISPLAYNAME = "name";
-    private static final String KEY_GMT = "gmt";
-    private static final String KEY_OFFSET = "offset";
+    private static final String KEY_ID = "id";  // value: String
+    private static final String KEY_DISPLAYNAME = "name";  // value: String
+    private static final String KEY_GMT = "gmt";  // value: String
+    private static final String KEY_OFFSET = "offset";  // value: int (Integer)
     private static final String XMLTAG_TIMEZONE = "timezone";
 
     private static final int HOURS_1 = 60 * 60000;
@@ -66,9 +66,6 @@ public class ZonePicker extends ListFragment {
     private static final int MENU_TIMEZONE = Menu.FIRST+1;
     private static final int MENU_ALPHABETICAL = Menu.FIRST;
 
-    // Initial focus position
-    private int mDefault;
-
     private boolean mSortedByTimezone;
 
     private SimpleAdapter mTimezoneSortedAdapter;
@@ -76,38 +73,73 @@ public class ZonePicker extends ListFragment {
 
     private ZoneSelectionListener mListener;
 
+    /**
+     * Constructs an adapter with TimeZone list. Sorted by TimeZone in default.
+     *
+     * @param sortedByName use Name for sorting the list.
+     */
+    public static SimpleAdapter constructTimezoneAdapter(Context context,
+            boolean sortedByName) {
+        final String[] from = new String[] {KEY_DISPLAYNAME, KEY_GMT};
+        final int[] to = new int[] {android.R.id.text1, android.R.id.text2};
+
+        final String sortKey = (sortedByName ? KEY_DISPLAYNAME : KEY_OFFSET);
+        final MyComparator comparator = new MyComparator(sortKey);
+        final List<HashMap<String, Object>> sortedList = getZones(context);
+        Collections.sort(sortedList, comparator);
+        final SimpleAdapter adapter = new SimpleAdapter(context,
+                sortedList,
+                android.R.layout.simple_list_item_2,
+                from,
+                to);
+
+        return adapter;
+    }
+
+    /**
+     * Searches {@link TimeZone} from the given {@link SimpleAdapter} object, and returns
+     * the index for the TimeZone.
+     *
+     * @param adapter SimpleAdapter constructed by
+     * {@link #constructTimezoneAdapter(Context, boolean)}.
+     * @param tz TimeZone to be searched.
+     * @return Index for the given TimeZone. -1 when there's no corresponding list item.
+     * returned.
+     */
+    public static int getTimeZoneIndex(SimpleAdapter adapter, TimeZone tz) {
+        final String defaultId = tz.getID();
+        final int listSize = adapter.getCount();
+        for (int i = 0; i < listSize; i++) {
+            // Using HashMap<String, Object> induces unnecessary warning.
+            final HashMap<?,?> map = (HashMap<?,?>)adapter.getItem(i);
+            final String id = (String)map.get(KEY_ID);
+            if (defaultId.equals(id)) {
+                // If current timezone is in this list, move focus to it
+                return i;
+            }
+        }
+        return -1;
+    }
+
+    /**
+     * @param item one of items in adapters. The adapter should be constructed by
+     * {@link #constructTimezoneAdapter(Context, boolean)}.
+     * @return TimeZone object corresponding to the item.
+     */
+    public static TimeZone obtainTimeZoneFromItem(Object item) {
+        return TimeZone.getTimeZone((String)((Map<?, ?>)item).get(KEY_ID));
+    }
+
     @Override
     public void onActivityCreated(Bundle savedInstanseState) {
         super.onActivityCreated(savedInstanseState);
 
-        final String[] from = new String[] {KEY_DISPLAYNAME, KEY_GMT};
-        final int[] to = new int[] {android.R.id.text1, android.R.id.text2};
-
-        MyComparator comparator = new MyComparator(KEY_OFFSET);
-
-        Activity activity = getActivity();
-        List<HashMap> timezoneSortedList = getZones();
-        Collections.sort(timezoneSortedList, comparator);
-        mTimezoneSortedAdapter = new SimpleAdapter(activity,
-                (List) timezoneSortedList,
-                android.R.layout.simple_list_item_2,
-                from,
-                to);
-
-        List<HashMap> alphabeticalList = new ArrayList<HashMap>(timezoneSortedList);
-        comparator.setSortingKey(KEY_DISPLAYNAME);
-        Collections.sort(alphabeticalList, comparator);
-        mAlphabeticalAdapter = new SimpleAdapter(getActivity(),
-                (List) alphabeticalList,
-                android.R.layout.simple_list_item_2,
-                from,
-                to);
+        final Activity activity = getActivity();
+        mTimezoneSortedAdapter = constructTimezoneAdapter(activity, false);
+        mAlphabeticalAdapter = constructTimezoneAdapter(activity, true);
 
         // Sets the adapter
         setSorting(true);
-
-        // If current timezone is in this list, move focus to it
-        setSelection(mDefault);
         setHasOptionsMenu(true);
     }
 
@@ -152,16 +184,22 @@ public class ZonePicker extends ListFragment {
         mListener = listener;
     }
 
-    private void setSorting(boolean timezone) {
-        setListAdapter(timezone ? mTimezoneSortedAdapter : mAlphabeticalAdapter);
-        mSortedByTimezone = timezone;
+    private void setSorting(boolean sortByTimezone) {
+        final SimpleAdapter adapter =
+                sortByTimezone ? mTimezoneSortedAdapter : mAlphabeticalAdapter;
+        setListAdapter(adapter);
+        mSortedByTimezone = sortByTimezone;
+        final int defaultIndex = getTimeZoneIndex(adapter, TimeZone.getDefault());
+        if (defaultIndex >= 0) {
+            setSelection(defaultIndex);
+        }
     }
 
-    private List<HashMap> getZones() {
-        List<HashMap> myData = new ArrayList<HashMap>();
-        long date = Calendar.getInstance().getTimeInMillis();
+    private static List<HashMap<String, Object>> getZones(Context context) {
+        final List<HashMap<String, Object>> myData = new ArrayList<HashMap<String, Object>>();
+        final long date = Calendar.getInstance().getTimeInMillis();
         try {
-            XmlResourceParser xrp = getActivity().getResources().getXml(R.xml.timezones);
+            XmlResourceParser xrp = context.getResources().getXml(R.xml.timezones);
             while (xrp.next() != XmlResourceParser.START_TAG)
                 continue;
             xrp.next();
@@ -192,15 +230,15 @@ public class ZonePicker extends ListFragment {
         return myData;
     }
 
-    protected void addItem(List<HashMap> myData, String id, String displayName, 
-            long date) {
-        HashMap map = new HashMap();
+    private static void addItem(
+            List<HashMap<String, Object>> myData, String id, String displayName, long date) {
+        final HashMap<String, Object> map = new HashMap<String, Object>();
         map.put(KEY_ID, id);
         map.put(KEY_DISPLAYNAME, displayName);
-        TimeZone tz = TimeZone.getTimeZone(id);
-        int offset = tz.getOffset(date);
-        int p = Math.abs(offset);
-        StringBuilder name = new StringBuilder();
+        final TimeZone tz = TimeZone.getTimeZone(id);
+        final int offset = tz.getOffset(date);
+        final int p = Math.abs(offset);
+        final StringBuilder name = new StringBuilder();
         name.append("GMT");
 
         if (offset < 0) {
@@ -223,20 +261,17 @@ public class ZonePicker extends ListFragment {
         map.put(KEY_GMT, name.toString());
         map.put(KEY_OFFSET, offset);
 
-        if (id.equals(TimeZone.getDefault().getID())) {
-            mDefault = myData.size();
-        }
-
         myData.add(map);
     }
 
     @Override
-    public void onListItemClick(ListView l, View v, int position, long id) {
-        Map map = (Map) l.getItemAtPosition(position);
+    public void onListItemClick(ListView listView, View v, int position, long id) {
+        final Map<?, ?> map = (Map<?, ?>)listView.getItemAtPosition(position);
+        final String tzId = (String) map.get(KEY_ID);
+
         // Update the system timezone value
         final Activity activity = getActivity();
-        AlarmManager alarm = (AlarmManager) activity.getSystemService(Context.ALARM_SERVICE);
-        String tzId = (String) map.get(KEY_ID);
+        final AlarmManager alarm = (AlarmManager) activity.getSystemService(Context.ALARM_SERVICE);
         alarm.setTimeZone(tzId);
         final TimeZone tz = TimeZone.getTimeZone(tzId);
         if (mListener != null) {
@@ -246,7 +281,7 @@ public class ZonePicker extends ListFragment {
         }
     }
 
-    private static class MyComparator implements Comparator<HashMap> {
+    private static class MyComparator implements Comparator<HashMap<?, ?>> {
         private String mSortingKey;
 
         public MyComparator(String sortingKey) {
@@ -257,7 +292,7 @@ public class ZonePicker extends ListFragment {
             mSortingKey = sortingKey;
         }
 
-        public int compare(HashMap map1, HashMap map2) {
+        public int compare(HashMap<?, ?> map1, HashMap<?, ?> map2) {
             Object value1 = map1.get(mSortingKey);
             Object value2 = map2.get(mSortingKey);
 

From 14ffda6f248d5a9d64c94ec6947ea317f75d6006 Mon Sep 17 00:00:00 2001
From: Dianne Hackborn <hackbod@google.com>
Date: Mon, 15 Nov 2010 18:02:06 -0800
Subject: [PATCH 3/4] Fix issue #3163240: Blank running services screen after
 returning from any specific service

Also get rid of async Vpn thing since the framework problem is fixed.

Change-Id: Ic7aafd666ea11b036e3e42c55332620db1fdf154
---
 .../settings/applications/ManageApplications.java        | 1 +
 src/com/android/settings/vpn/VpnSettings.java            | 9 +--------
 2 files changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/com/android/settings/applications/ManageApplications.java b/src/com/android/settings/applications/ManageApplications.java
index 94283ee2845..be5019b93a0 100644
--- a/src/com/android/settings/applications/ManageApplications.java
+++ b/src/com/android/settings/applications/ManageApplications.java
@@ -580,6 +580,7 @@ public class ManageApplications extends Fragment implements
                 R.id.running_processes);
 
         mCreatedRunning = mResumedRunning = false;
+        mCurView = VIEW_NOTHING;
         
         View tabRoot = mInflater.inflate(com.android.internal.R.layout.tab_content, null);
         mTabHost = (TabHost)tabRoot.findViewById(com.android.internal.R.id.tabhost);
diff --git a/src/com/android/settings/vpn/VpnSettings.java b/src/com/android/settings/vpn/VpnSettings.java
index 0b77930670c..4dfe21a0a0b 100644
--- a/src/com/android/settings/vpn/VpnSettings.java
+++ b/src/com/android/settings/vpn/VpnSettings.java
@@ -138,8 +138,6 @@ public class VpnSettings extends SettingsPreferenceFragment
 
     private StatusChecker mStatusChecker = new StatusChecker();
 
-    private Handler mHandler = new Handler();
-
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
@@ -392,12 +390,7 @@ public class VpnSettings extends SettingsPreferenceFragment
 
         if (requestCode == REQUEST_SELECT_VPN_TYPE) {
             final String typeName = data.getStringExtra(KEY_VPN_TYPE);
-            mHandler.post(new Runnable() {
-
-                public void run() {
-                    startVpnEditor(createVpnProfile(typeName), true);
-                }
-            });
+            startVpnEditor(createVpnProfile(typeName), true);
         } else if (requestCode == REQUEST_ADD_OR_EDIT_PROFILE) {
             VpnProfile p = data.getParcelableExtra(KEY_VPN_PROFILE);
             if (p == null) {

From 4dd2a5b4f7129609d773a7e663c48f81b89a60b2 Mon Sep 17 00:00:00 2001
From: Daisuke Miyakawa <dmiyakawa@google.com>
Date: Fri, 12 Nov 2010 12:56:56 -0800
Subject: [PATCH 4/4] Use Handler to delay enabling skip button.

- enable skip button after a while
- stop ignoring wifi status

Bug: 3186913
Change-Id: Ia565f77ecb0d11ad514548774b49d8eda6db4ff8
---
 .../wifi/WifiSettingsForSetupWizardXL.java    | 141 ++++++++----------
 1 file changed, 62 insertions(+), 79 deletions(-)

diff --git a/src/com/android/settings/wifi/WifiSettingsForSetupWizardXL.java b/src/com/android/settings/wifi/WifiSettingsForSetupWizardXL.java
index 798ec1211bb..1781847ba0b 100644
--- a/src/com/android/settings/wifi/WifiSettingsForSetupWizardXL.java
+++ b/src/com/android/settings/wifi/WifiSettingsForSetupWizardXL.java
@@ -23,6 +23,7 @@ import android.content.Context;
 import android.net.NetworkInfo.DetailedState;
 import android.net.wifi.WifiManager;
 import android.os.Bundle;
+import android.os.Handler;
 import android.preference.PreferenceCategory;
 import android.text.TextUtils;
 import android.util.Log;
@@ -44,6 +45,7 @@ import java.util.EnumMap;
  */
 public class WifiSettingsForSetupWizardXL extends Activity implements OnClickListener {
     private static final String TAG = "SetupWizard";
+    private static final boolean DEBUG = true;
 
     private static final EnumMap<DetailedState, DetailedState> stateMap =
             new EnumMap<DetailedState, DetailedState>(DetailedState.class);
@@ -80,25 +82,27 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
 
     // true when a user already pressed "Connect" button and waiting for connection.
     // Also true when the device is already connected to a wifi network on launch.
-    private boolean mAfterTryConnect;
+    private boolean mAfterConnectAction;
 
     private WifiConfigUiForSetupWizardXL mWifiConfig;
 
     private InputMethodManager mInputMethodManager;
 
-    // This count reduces every time when there's a notification about WiFi status change.
-    // During the term this is >0, The system refrains some actions which are not appropriate
-    // at that timing.
-    // - When network is connected at that timing, this screen doesn't call finish().
-    //   This count is set to 0 when being detected (not decremente).
-    // - When network status is "disconnected", we just show the message "connecting"
-    //   regardless of the actual WiFi status.
-    //   (After this count's becoming 0, the status message correctly reflects what WiFi Picker
-    //    told it)
-    // This is a tweak for letting users not confused with WiFi state during a first first steps.
-    private int mIgnoringWifiNotificationCount = 5;
+    private final Handler mHandler = new Handler();
 
-    private boolean mShowingConnectingMessageManually = false;
+    // At first, we set "Skip" button disabled so that users won't press it soon after the screen
+    // migration. The button is enabled after the wifi module returns some result
+    // (a list of available network, etc.) One possible problem is that the notification from the
+    // wifi module may be delayed and users may be stuck here, without any other way to exit this
+    // screen.
+    // To let users exit this Activity, we enable the button after waiting for a moment.
+    private final int DELAYED_SKIP_ENABLE_TIME = 10000;  // Unit: millis
+    private final Runnable mSkipButtonEnabler = new Runnable() {
+        public void run() {
+            if (DEBUG) Log.d(TAG, "Delayed skip enabler starts running.");
+            mSkipOrNextButton.setEnabled(true);
+        }
+    };
 
     @Override
     public void onCreate(Bundle savedInstanceState) {
@@ -145,6 +149,11 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
         mBackButton.setOnClickListener(this);
         mDetailButton = (Button)findViewById(R.id.wifi_setup_detail);
         mDetailButton.setOnClickListener(this);
+
+        // At first, Wifi module doesn't return SCANNING state (it's too early), so we manually
+        // show it.
+        showScanningStatus();
+        mHandler.postDelayed(mSkipButtonEnabler, DELAYED_SKIP_ENABLE_TIME);
     }
 
     private void restoreFirstButtonVisibilityState() {
@@ -161,10 +170,13 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
     public void onClick(View view) {
         hideSoftwareKeyboard();
         if (view == mAddNetworkButton) {
+            if (DEBUG) Log.d(TAG, "AddNetwork button pressed");
             onAddNetworkButtonPressed();
         } else if (view == mRefreshButton) {
+            if (DEBUG) Log.d(TAG, "Refresh button pressed");
             refreshAccessPoints(true);
         } else if (view == mSkipOrNextButton) {
+            if (DEBUG) Log.d(TAG, "Skip/Next button pressed");
             if (TextUtils.equals(getString(R.string.wifi_setup_skip), ((Button)view).getText())) {
                 // We don't want to let Wifi enabled when a user press skip without choosing
                 // any access point.
@@ -173,12 +185,16 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
             setResult(Activity.RESULT_OK);
             finish();            
         } else if (view == mConnectButton) {
+            if (DEBUG) Log.d(TAG, "Connect button pressed");
             onConnectButtonPressed();
         } else if (view == mForgetButton) {
+            if (DEBUG) Log.d(TAG, "Forget button pressed");
             onForgetButtonPressed();
         } else if (view == mBackButton) {
+            if (DEBUG) Log.d(TAG, "Back button pressed");
             onBackButtonPressed();
         } else if (view == mDetailButton) {
+            if (DEBUG) Log.d(TAG, "Detail button pressed");
             mWifiSettings.showDialogForSelectedPreference();
         }
     }
@@ -216,11 +232,14 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
             break;
         }
         case CONNECTING: {
-            mShowingConnectingMessageManually = false;
             showConnectingStatus();
             break;
         }
         case CONNECTED: {
+            // If the device is already connected to a wifi without users' "Connect" request,
+            // this can be false here. We want to treat it as "after connect action".
+            mAfterConnectAction = true;
+
             mProgressBar.setIndeterminate(false);
             mProgressBar.setProgress(2);
             mProgressText.setText(Summary.get(this, state));
@@ -231,66 +250,29 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
             mBackButton.setVisibility(View.VISIBLE);
             mSkipOrNextButton.setVisibility(View.VISIBLE);
             mSkipOrNextButton.setEnabled(true);
+            mHandler.removeCallbacks(mSkipButtonEnabler);
 
-            if (mIgnoringWifiNotificationCount > 0) {
-                // The network is already available before doing anything. We avoid skip this
-                // screen to avoid unnecessary trouble by doing so.
-                mIgnoringWifiNotificationCount = 0;
-                mAfterTryConnect = true;
-            } else {
-                mProgressText.setText(Summary.get(this, state));
-                // setResult(Activity.RESULT_OK);
-                // finish();
-            }
-            break;
-        }
-        case FAILED: {
-            mProgressBar.setIndeterminate(false);
-            mProgressBar.setProgress(0);
-            mStatusText.setText(R.string.wifi_setup_status_select_network);
             mProgressText.setText(Summary.get(this, state));
-            
-            restoreFirstButtonVisibilityState();
-            mAddNetworkButton.setEnabled(true);
-            mRefreshButton.setEnabled(true);
-            mSkipOrNextButton.setEnabled(true);
             break;
         }
-        default:  // Not connected.
-            if (mWifiSettings.getAccessPointsCount() == 0 && mIgnoringWifiNotificationCount > 0) {
-                Log.d(TAG, "Currently not connected, but we show \"Scanning\" for a moment");
-                mIgnoringWifiNotificationCount--;
-                mProgressBar.setIndeterminate(true);
-                mProgressText.setText(Summary.get(this, DetailedState.SCANNING));
-            } else if (mShowingConnectingMessageManually && mIgnoringWifiNotificationCount > 0) {
-                Log.i(TAG, "Currently not connected, but we show \"connecting\" for a moment.");
-                mIgnoringWifiNotificationCount--;
-                showConnectingStatus();
-            } else {
-                if (mAfterTryConnect) {
-                    // TODO: how to stop connecting the network?
-                    Log.i(TAG, String.format(
-                            "State %s has been notified after trying to connect a network. ",
-                            state.toString()));
-                }
-
-
-                mShowingConnectingMessageManually = false;
-                mProgressBar.setIndeterminate(false);
-                mProgressBar.setProgress(0);
-
-                mStatusText.setText(R.string.wifi_setup_not_connected);
-                mProgressText.setText(getString(R.string.wifi_setup_not_connected));
-
-                mAddNetworkButton.setEnabled(true);
-                mRefreshButton.setEnabled(true);
-                mSkipOrNextButton.setEnabled(true);
-            }
-
+        default:  // DISCONNECTED, FAILED
+            showDisconnectedStatus(Summary.get(this, state));
             break;
         }
     }
 
+    private void showDisconnectedStatus(String stateString) {
+        mProgressBar.setIndeterminate(false);
+        mProgressBar.setProgress(0);
+
+        mStatusText.setText(stateString);
+        mProgressText.setText(stateString);
+
+        mAddNetworkButton.setEnabled(true);
+        mRefreshButton.setEnabled(true);
+        mSkipOrNextButton.setEnabled(true);
+    }
+
     private void showConnectingStatus() {
         mProgressBar.setIndeterminate(false);
         mProgressBar.setProgress(1);
@@ -298,6 +280,14 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
         mProgressText.setText(Summary.get(this, DetailedState.CONNECTING));
     }
 
+    private void showScanningStatus() {
+        mProgressBar.setIndeterminate(true);
+        ((Button)findViewById(R.id.wifi_setup_add_network)).setEnabled(false);
+        ((Button)findViewById(R.id.wifi_setup_refresh_list)).setEnabled(false);
+        mProgressText.setText(Summary.get(this, DetailedState.SCANNING));
+        mStatusText.setText(R.string.wifi_setup_status_scanning);
+    }
+
     private void onAddNetworkButtonPressed() {
         // onConfigUiShown() will be called.
         mWifiSettings.onAddNetworkPressed();
@@ -345,7 +335,7 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
 
     // May be called when user press "connect" button in WifiDialog
     /* package */ void onConnectButtonPressed() {
-        mAfterTryConnect = true;
+        mAfterConnectAction = true;
 
         mWifiSettings.submit(mWifiConfig.getController());
 
@@ -368,9 +358,6 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
         mAddNetworkButton.setVisibility(View.GONE);
         mRefreshButton.setVisibility(View.GONE);
         mDetailButton.setVisibility(View.GONE);
-
-        mShowingConnectingMessageManually = true;
-        mIgnoringWifiNotificationCount = 1;
     }
 
     // May be called when user press "forget" button in WifiDialog
@@ -389,17 +376,18 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
     }
 
     private void onBackButtonPressed() {
-        if (mAfterTryConnect) {
-            mAfterTryConnect = false;
+        if (mAfterConnectAction) {
+            if (DEBUG) Log.d(TAG, "Back button pressed after connect action.");
+            mAfterConnectAction = false;
 
             // When a user press "Back" button after pressing "Connect" button, we want to cancel
             // the "Connect" request and refresh the whole wifi status.
             restoreFirstButtonVisibilityState();
-            mShowingConnectingMessageManually = false;
 
             mAddNetworkButton.setEnabled(false);
             mRefreshButton.setEnabled(false);
             mSkipOrNextButton.setEnabled(true);
+            changeNextButtonState(false);  // Skip
 
             refreshAccessPoints(true);
         } else { // During user's Wifi configuration.
@@ -444,12 +432,7 @@ public class WifiSettingsForSetupWizardXL extends Activity implements OnClickLis
     }
 
     private void refreshAccessPoints(boolean disconnectNetwork) {
-        mIgnoringWifiNotificationCount = 5;
-        mProgressBar.setIndeterminate(true);
-        ((Button)findViewById(R.id.wifi_setup_add_network)).setEnabled(false);
-        ((Button)findViewById(R.id.wifi_setup_refresh_list)).setEnabled(false);
-        mProgressText.setText(Summary.get(this, DetailedState.SCANNING));
-        mStatusText.setText(R.string.wifi_setup_status_scanning);
+        showScanningStatus();
 
         if (disconnectNetwork) {
             mWifiManager.disconnect();