Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1505)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java

Issue 440283002: Android: Get AutofillProfile summary labels from PersonalDataManagerAndroid (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: after early return Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java b/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
index 919a9d757df4273013305173cdebc8a6504e8b47..141b50a1cd5af01db7cce1dcc75d0c1ddeac7da0 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/autofill/PersonalDataManager.java
@@ -50,6 +50,7 @@ public class PersonalDataManager {
private String mCountry;
private String mPhoneNumber;
private String mEmailAddress;
+ private String mLabel;
private String mLanguageCode;
@CalledByNative("AutofillProfile")
@@ -126,6 +127,10 @@ public class PersonalDataManager {
return mDependentLocality;
}
+ public String getLabel() {
+ return mLabel;
+ }
+
@CalledByNative("AutofillProfile")
public String getPostalCode() {
return mPostalCode;
@@ -164,6 +169,10 @@ public class PersonalDataManager {
mGUID = guid;
}
+ public void setLabel(String label) {
+ mLabel = label;
+ }
+
public void setOrigin(String origin) {
mOrigin = origin;
}
@@ -363,11 +372,17 @@ public class PersonalDataManager {
public List<AutofillProfile> getProfiles() {
ThreadUtils.assertOnUiThread();
+
+ String[] profileLabels = nativeGetProfileLabels(mPersonalDataManagerAndroid);
+
int profileCount = nativeGetProfileCount(mPersonalDataManagerAndroid);
List<AutofillProfile> profiles = new ArrayList<AutofillProfile>(profileCount);
for (int i = 0; i < profileCount; i++) {
- profiles.add(nativeGetProfileByIndex(mPersonalDataManagerAndroid, i));
+ AutofillProfile profile = nativeGetProfileByIndex(mPersonalDataManagerAndroid, i);
+ profile.setLabel(profileLabels[i]);
+ profiles.add(profile);
}
+
return profiles;
}
@@ -435,6 +450,7 @@ public class PersonalDataManager {
private native long nativeInit();
private native int nativeGetProfileCount(long nativePersonalDataManagerAndroid);
+ private native String[] nativeGetProfileLabels(long nativePersonalDataManagerAndroid);
private native AutofillProfile nativeGetProfileByIndex(long nativePersonalDataManagerAndroid,
int index);
private native AutofillProfile nativeGetProfileByGUID(long nativePersonalDataManagerAndroid,
« no previous file with comments | « no previous file | chrome/android/javatests/src/org/chromium/chrome/browser/autofill/AutofillTestHelper.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698