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

base/android/jni_local_ref_extensions.h

+// 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.
+
+#ifndef BASE_ANDROID_JNI_LOCAL_REF_EXTENSIONS_H_
+#define BASE_ANDROID_JNI_LOCAL_REF_EXTENSIONS_H_
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_array.h"
+#include "base/android/jni_string.h"
+
+namespace base {
+namespace android {
+
+// Returns empty string if java string is null,

[Ilya Sherman, 2013/03/05 00:42:38]

nit: "Returns and empty string if the Java string is null,"

[apiccion, 2013/03/09 00:53:41]

Done.

+// otherwise converts and returns the string.
+string16 SafeJavaStringToUTF16(ScopedJavaLocalRef<jstring> jstring);
+
+// Checks if java array is null and copies contents into stringVector

[Ilya Sherman, 2013/03/05 00:42:38]

nit: Perhaps something like "Fills |string_vector| with the contents of
|jarray|.  If the Java array is null, then |string_vector| is not modified."

[apiccion, 2013/03/09 00:53:41]

Done.

+void SafeJavaStringArrayToStringVector(
+  ScopedJavaLocalRef<jobjectArray> jarray,
+  JNIEnv* env,
+  std::vector<string16>* stringVector);

[Ilya Sherman, 2013/03/05 00:42:38]

nit: Use hacker_case, not camelCase.

[apiccion, 2013/03/09 00:53:41]

Done.

+
+}  // namespace android
+}  // namespace base
+
+#endif  // BASE_ANDROID_JNI_LOCAL_REF_EXTENSIONS_H_