Support datalist for date/time input types on Android

content/public/android/java/src/org/chromium/content/browser/input/InputDialogContainer.java

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.content.browser.input;
 
 import android.app.AlertDialog;
+import android.app.AlertDialog;

[Miguel Garcia, 2013/10/08 17:44:40]

double import

[keishi, 2013/10/21 17:00:58]

Done.

 import android.app.DatePickerDialog;
 import android.app.TimePickerDialog;
 import android.app.DatePickerDialog.OnDateSetListener;
 import android.app.TimePickerDialog.OnTimeSetListener;
 import android.content.Context;
 import android.content.DialogInterface;
 import android.content.DialogInterface.OnDismissListener;
 import android.text.TextUtils;
 import android.text.format.DateFormat;
 import android.text.format.Time;
+import android.view.LayoutInflater;
+import android.view.View;
+import android.widget.AdapterView;
 import android.widget.DatePicker;
+import android.widget.ListView;
+import android.widget.SimpleAdapter;
+import android.widget.TextView;
 import android.widget.TimePicker;
 
 import org.chromium.content.browser.input.DateTimePickerDialog.OnDateTimeSetListener;
 import org.chromium.content.browser.input.MultiFieldTimePickerDialog.OnMultiFieldTimeSetListener;
 import org.chromium.content.browser.input.TwoFieldDatePickerDialog;
 import org.chromium.content.R;
 
+import java.util.Arrays;
 import java.text.ParseException;
 import java.text.SimpleDateFormat;
 import java.util.Calendar;
 import java.util.Date;
+import java.util.List;
+import java.util.Map;
+import java.util.HashMap;
+import java.util.ArrayList;
 
 public class InputDialogContainer {
 
     interface InputActionDelegate {
         void cancelDateTimeDialog();
         void replaceDateTime(int dialogType,
             int year, int month, int day, int hour, int minute, int second, int milli, int week);
+        void acceptDataListSuggestion(String value);
     }
 
     // Default values used in Time representations of selected date/time before formatting.
     // They are never displayed to the user.
     private static final int YEAR_DEFAULT = 1970;
     private static final int MONTH_DEFAULT = 0;
     private static final int MONTHDAY_DEFAULT = 1;
     private static final int HOUR_DEFAULT = 0;
     private static final int MINUTE_DEFAULT = 0;
     private static final int WEEK_DEFAULT = 0;
@@ skipping 54 matching lines @@
             Calendar cal = Calendar.getInstance();
             result.set(cal.get(Calendar.SECOND), cal.get(Calendar.MINUTE),
                     cal.get(Calendar.HOUR_OF_DAY), cal.get(Calendar.DATE),
                     cal.get(Calendar.MONTH), cal.get(Calendar.YEAR));
         } else {
             result.set(second, minute, hour, monthDay, month, year);
         }
         return result;
     }
 
+    void showSuggestionDialog(final int dialogType,

[Miguel Garcia, 2013/10/08 17:44:40]

Add javadoc.
Can this be private?

I think this should be moved to it's own class, something called
DateTimeSuggestionDialog with a show method or something like that.

+            final int year, final int month, final int monthDay,
+            final int hour, final int minute, final int second, final int milli, final int week,
+            final double min, final double max, final double step,
+            DateTimeSuggestion[] suggestions) {
+        if (isDialogShowing()) mDialog.dismiss();
+
+        ListView suggestionListView = new ListView(mContext);
+        LayoutInflater inflater =

[newt (away), 2013/10/09 07:20:52]

unused variable

[keishi, 2013/10/21 17:00:58]

Done.

+            (LayoutInflater) mContext.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
+        ArrayList<DateTimeSuggestion> suggestionList =
+            new ArrayList<DateTimeSuggestion>(Arrays.asList(suggestions));
+        final DateTimeSuggestionListAdapter adapter =
+            new DateTimeSuggestionListAdapter(mContext, suggestionList);

[Miguel Garcia, 2013/10/08 17:44:40]

nit: no need for the suggestionList temp variable, just do
final DateTimeSuggestionListAdapter adapter = new
DateTimeSuggestionListAdapter(mContext, Arrays.asList(suggestions));

[keishi, 2013/10/21 17:00:58]

Done.

+        suggestionListView.setAdapter(adapter);
+        suggestionListView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
+            @Override
+            public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
+                if (position == adapter.getCount() - 1) {
+                    showPickerDialog(dialogType, year, month, monthDay,
+                                     hour, minute, second, milli,
+                                     week, min, max, step);
+                } else {
+                    mInputActionDelegate.acceptDataListSuggestion(adapter.getItem(position).mValue);

[Miguel Garcia, 2013/10/08 17:44:40]

can you call replaceDateTime with the right values instead of adding another
method?

[keishi, 2013/10/21 17:00:58]

Done.

+                    mDialogAlreadyDismissed = true;
+                }
+            }
+        });
+
+        int dialogTitleId = R.string.date_picker_dialog_title;

[Miguel Garcia, 2013/10/08 17:44:40]

How does this layout on a phone, it seems the design will work on tablets but
with smaller screens it will probably overflow. Can you send a screenshot on
portrait mode on a phone?

[keishi, 2013/10/21 17:00:58]

I don't have a device with a small screen so here is a screenshot with very long
strings. I think it looks good.
https://docs.google.com/file/d/0BwRi59l_ri74bExaTGg4aFE3NzQ/edit?usp=sharing

+        if (dialogType == sTextInputTypeTime) {
+            dialogTitleId = R.string.time_picker_dialog_title;
+        } else if (dialogType == sTextInputTypeDateTime ||
+                dialogType == sTextInputTypeDateTimeLocal) {
+            dialogTitleId = R.string.date_time_picker_dialog_title;
+        } else if (dialogType == sTextInputTypeMonth) {
+            dialogTitleId = R.string.month_picker_dialog_title;
+        } else if (dialogType == sTextInputTypeWeek) {
+            dialogTitleId = R.string.week_picker_dialog_title;
+        }
+
+        mDialog = new AlertDialog.Builder(mContext)
+            .setTitle(dialogTitleId)
+            .setView(suggestionListView)
+            .setOnDismissListener(new DialogInterface.OnDismissListener() {
+                @Override
+                public void onDismiss(DialogInterface dialog) {
+                    if (mDialog == dialog && !mDialogAlreadyDismissed) {
+                        mDialogAlreadyDismissed = true;
+                        mInputActionDelegate.cancelDateTimeDialog();
+                    }
+                }
+            })
+            .setNegativeButton(mContext.getText(android.R.string.cancel),
+                new DialogInterface.OnClickListener() {
+                    @Override
+                    public void onClick(DialogInterface dialog, int which) {
+                        mDialogAlreadyDismissed = true;

[newt (away), 2013/10/09 07:20:52]

Can you replace lines 181-182 with "mDialog.dismiss()"?

[keishi, 2013/10/21 17:00:58]

Done.

+                        mInputActionDelegate.cancelDateTimeDialog();
+                    }
+                })
+            .create();
+
+        mDialogAlreadyDismissed = false;
+        mDialog.show();
+    }
+
     void showDialog(final int dialogType, int year, int month, int monthDay,
                     int hour, int minute, int second, int milli, int week,
-                    double min, double max, double step) {
+                    double min, double max, double step,
+                    DateTimeSuggestion[] suggestions) {
+        if (suggestions == null) {
+            showPickerDialog(dialogType, year, month, monthDay,
+                             hour, minute, second, milli,
+                             week, min, max, step);
+        } else {
+            showSuggestionDialog(dialogType, year, month, monthDay,
+                                 hour, minute, second, milli, week,
+                                 min, max, step, suggestions);
+        }
+    }
+
+    void showPickerDialog(final int dialogType, int year, int month, int monthDay,
+                          int hour, int minute, int second, int milli, int week,
+                          double min, double max, double step) {
         if (isDialogShowing()) mDialog.dismiss();

[Miguel Garcia, 2013/10/08 17:44:40]

can you move this check to the showDialog call instead of repeating it in both
showPickerDialog and showSuggestionDialog?

[keishi, 2013/10/21 17:00:58]

Done.

 
         // Java Date dialogs like longs but Blink prefers doubles..
         // Both parameters mean different things depending on the type
         // For input type=month min and max come as number on months since 1970
         // For other types (including type=time) they are just milliseconds since 1970
         // In any case the cast here is safe given the above restrictions.
         long minTime = (long) min;
         long maxTime = (long) max;
         int stepTime = (int) step;
 
@@ skipping 186 matching lines @@
             int year, int month, int monthDay, int hourOfDay,
             int minute, int second, int milli, int week, String dateFormat) {
         // Prevents more than one callback being sent to the native
         // side when the dialog triggers multiple events.
         mDialogAlreadyDismissed = true;
 
         mInputActionDelegate.replaceDateTime(
             dialogType, year, month, monthDay, hourOfDay, minute, second, milli, week);
     }
 }