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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java

Issue 2163693002: [Merge M-53] Credit card editor for PaymentRequest UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2785
Patch Set: Created 4 years, 5 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
Index: chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java b/chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java
index a5fb8fa3fc56e71ac92b19cbb4a0a12f466967c6..bade26fc6d6b632a2d9da00cf70c9e8ae21ca15c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/payments/ContactEditor.java
@@ -84,19 +84,19 @@ public class ContactEditor extends EditorBase<AutofillContact> {
? new AutofillContact(new AutofillProfile(), null, null, false) : toEdit;
final EditorFieldModel phoneField = mRequestPayerPhone
- ? new EditorFieldModel(EditorFieldModel.INPUT_TYPE_HINT_PHONE,
+ ? EditorFieldModel.createTextInput(EditorFieldModel.INPUT_TYPE_HINT_PHONE,
mContext.getString(R.string.autofill_profile_editor_phone_number),
mPhoneNumbers, getPhoneValidator(),
- mContext.getString(R.string.payments_phone_required_validation_message),
+ mContext.getString(R.string.payments_field_required_validation_message),
mContext.getString(R.string.payments_phone_invalid_validation_message),
contact.getPayerPhone())
: null;
final EditorFieldModel emailField = mRequestPayerEmail
- ? new EditorFieldModel(EditorFieldModel.INPUT_TYPE_HINT_EMAIL,
+ ? EditorFieldModel.createTextInput(EditorFieldModel.INPUT_TYPE_HINT_EMAIL,
mContext.getString(R.string.autofill_profile_editor_email_address),
mEmailAddresses, getEmailValidator(),
- mContext.getString(R.string.payments_email_required_validation_message),
+ mContext.getString(R.string.payments_field_required_validation_message),
mContext.getString(R.string.payments_email_invalid_validation_message),
contact.getPayerEmail())
: null;
@@ -130,8 +130,8 @@ public class ContactEditor extends EditorBase<AutofillContact> {
contact.getProfile().setEmailAddress(email);
}
- PersonalDataManager.getInstance().setProfile(contact.getProfile());
- contact.completeContact(phone, email);
+ String guid = PersonalDataManager.getInstance().setProfile(contact.getProfile());
+ contact.completeContact(guid, phone, email);
callback.onResult(contact);
}
});

Powered by Google App Engine
This is Rietveld 408576698