Support datalist for date/time input types on Android

content/browser/android/date_time_chooser_android.cc

 // Copyright (c) 2013 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.
 
 #include "content/browser/android/date_time_chooser_android.h"
 
+#include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
+#include "content/common/date_time_suggestion.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/android/content_view_core.h"
 #include "content/public/browser/render_view_host.h"
 #include "jni/DateTimeChooserAndroid_jni.h"
+#include "third_party/icu/source/common/unicode/uchar.h"
 
 using base::android::AttachCurrentThread;
 using base::android::ConvertJavaStringToUTF16;
 using base::android::ConvertUTF8ToJavaString;
+using base::android::ConvertUTF16ToJavaString;
 
 
+namespace {
+
+bool IsNonPrintableChar(char16 c) {
+  return !u_isprint(static_cast<UChar>(c));
+}
+
+string16 SanitizeSuggestionString(string16 string) {

[bulach, 2013/12/03 14:22:29]

nit: parameter as const&?

[keishi, 2013/12/03 16:52:34]

Done.

+  string.erase(
+          std::remove_if(string.begin(), string.end(), &IsNonPrintableChar),

[bulach, 2013/12/03 14:22:29]

shouldn't this be using base::i18n::UTF16CharIterator instead?
if not, nit: wrong indent :)

[keishi, 2013/12/03 16:52:34]

Done.

+          string.end());
+  return string.substr(0, 255);
+}
+
+}  // namespace
+
 namespace content {
 
 // DateTimeChooserAndroid implementation
 DateTimeChooserAndroid::DateTimeChooserAndroid()
   : host_(NULL) {
 }
 
 DateTimeChooserAndroid::~DateTimeChooserAndroid() {
 }
 
@@ skipping 20 matching lines @@
   host_->Send(new ViewMsg_CancelDateTimeDialog(host_->GetRoutingID()));
 }
 
 void DateTimeChooserAndroid::ShowDialog(
     ContentViewCore* content,
     RenderViewHost* host,
     ui::TextInputType dialog_type,
     double dialog_value,
     double min,
     double max,
-    double step) {
+    double step,
+    std::vector<DateTimeSuggestion> suggestions) {
   host_ = host;
 
   JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobjectArray> suggestions_array;
+
+  if (suggestions.size() > 0) {
+    ScopedJavaLocalRef<jclass> suggestion_class = base::android::GetClass(
+        env,
+        "org/chromium/content/browser/input/DateTimeSuggestion");

[bulach, 2013/12/03 14:22:29]

we try as much as possible to use GetClass, NewObjectArray, etc... directly...

this will be *much* clearer as:

ScopedJavaLocalRef<jobject> j_suggestions =
Java_DateTimeChooserAndroid_createSuggestionsArray(env, suggestions.size);
...
then in 97, rather than env->SetObjectArrayElement, call:


Java_DateTimeChooserAndroid_setDateTimeSuggestionAt(env,
   j_suggestions, i, suggestion.value, localized_value.obj(), label.obj());

[keishi, 2013/12/03 16:52:34]

Done.

+    suggestions_array.Reset(env, env->NewObjectArray(suggestions.size(),
+                                                     suggestion_class.obj(),
+                                                     NULL));
+    for (size_t i = 0; i < suggestions.size(); ++i) {
+      const content::DateTimeSuggestion& suggestion = suggestions[i];
+      ScopedJavaLocalRef<jstring> localizedValue = ConvertUTF16ToJavaString(

[bulach, 2013/12/03 14:22:29]

nit: localized_value

[keishi, 2013/12/03 16:52:34]

Done.

+          env, SanitizeSuggestionString(suggestion.localizedValue));
+      ScopedJavaLocalRef<jstring> label = ConvertUTF16ToJavaString(
+          env, SanitizeSuggestionString(suggestion.label));
+      env->SetObjectArrayElement(
+          suggestions_array.obj(),
+          i,
+          Java_DateTimeChooserAndroid_createDateTimeSuggestion(
+              env,
+              suggestion.value,
+              localizedValue.obj(),
+              label.obj()).obj());
+    }
+  }
+
   j_date_time_chooser_.Reset(Java_DateTimeChooserAndroid_createDateTimeChooser(
       env,
       content->GetJavaObject().obj(),
       reinterpret_cast<intptr_t>(this),
       dialog_type,
       dialog_value,
       min,
       max,
-      step));
+      step,
+      suggestions_array.obj()));
 }
 
 // ----------------------------------------------------------------------------
 // Native JNI methods
 // ----------------------------------------------------------------------------
 bool RegisterDateTimeChooserAndroid(JNIEnv* env) {
   bool registered = RegisterNativesImpl(env);
   if (registered)
     DateTimeChooserAndroid::InitializeDateInputTypes(
         ui::TEXT_INPUT_TYPE_DATE,
         ui::TEXT_INPUT_TYPE_DATE_TIME,
         ui::TEXT_INPUT_TYPE_DATE_TIME_LOCAL,
         ui::TEXT_INPUT_TYPE_MONTH,
         ui::TEXT_INPUT_TYPE_TIME,
         ui::TEXT_INPUT_TYPE_WEEK);
   return registered;
 }
 
 }  // namespace content