Added personal_data_manager android implementation for auto-populating auto-fill on android builds …

base/android/jni_local_ref_extensions.cc

Index: base/android/jni_local_ref_extensions.cc
diff --git a/base/android/jni_local_ref_extensions.cc b/base/android/jni_local_ref_extensions.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2154ad96e7bf00659dba6149d33e468b086515e2
--- /dev/null
+++ b/base/android/jni_local_ref_extensions.cc
@@ -0,0 +1,31 @@
+// 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 "base/android/jni_local_ref_extensions.h"
+
+namespace base {
+namespace android {
+
+string16 SafeJavaStringToUTF16(ScopedJavaLocalRef<jstring> jstring) {

[Ilya Sherman, 2013/03/01 01:55:04]

nit: Please make the order of definitions in the implementation file match the
order of declarations in the header file.

[apiccion, 2013/03/02 03:37:01]

Done.

+  if (!jstring.is_null()) {
+    return ConvertJavaStringToUTF16(jstring);
+  } else {
+    return string16();
+  }

[Ilya Sherman, 2013/03/01 01:55:04]

nit: Please write this function like so:

if (jstring.is_null())
  return string16();

return ConvertJavaStringToUTF16(jstring);

[apiccion, 2013/03/02 03:37:01]

Done.

+}
+
+std::vector<string16> SafeJavaStringArrayToStringVector(
+  ScopedJavaLocalRef<jobjectArray> jarray,
+  JNIEnv* env) {
+  std::vector<string16> stringVector;
+  if (!jarray.is_null()) {
+    base::android::AppendJavaStringArrayToStringVector(env,
+                                                       jarray.obj(),
+                                                       &stringVector);
+  }
+  return stringVector;
+}
+
+}  // namespace android
+}  // namespace base