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

Unified Diff: ios/chrome/browser/payments/payment_request_view_controller.mm

Issue 2588913002: EarlGrey tests for Payment Request (base CL) (Closed)
Patch Set: Moved the experimental flag back to the BVC Created 3 years, 11 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: ios/chrome/browser/payments/payment_request_view_controller.mm
diff --git a/ios/chrome/browser/payments/payment_request_view_controller.mm b/ios/chrome/browser/payments/payment_request_view_controller.mm
index 6e9a14ea6833adf17a722387c52fcaa4cd46283d..2be5dc7c276dfb7f6cd12df9532195c53dd42854 100644
--- a/ios/chrome/browser/payments/payment_request_view_controller.mm
+++ b/ios/chrome/browser/payments/payment_request_view_controller.mm
@@ -14,6 +14,7 @@
#include "components/autofill/core/browser/credit_card.h"
#include "components/autofill/core/browser/field_types.h"
#include "components/autofill/core/browser/personal_data_manager.h"
+#include "components/strings/grit/components_strings.h"
#include "ios/chrome/browser/application_context.h"
#import "ios/chrome/browser/payments/cells/page_info_item.h"
#import "ios/chrome/browser/payments/cells/payment_method_item.h"
@@ -34,6 +35,10 @@
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
+using payment_request_utils::NameLabelFromAutofillProfile;
+using payment_request_utils::AddressLabelFromAutofillProfile;
+using payment_request_utils::PhoneNumberLabelFromAutofillProfile;
+
NSString* const kPaymentRequestCollectionViewId =
@"kPaymentRequestCollectionViewId";
@@ -111,6 +116,8 @@ typedef NS_ENUM(NSInteger, ItemType) {
NSForegroundColorAttributeName : [UIColor lightGrayColor]
}
forState:UIControlStateDisabled];
+ [_cancelButton
+ setAccessibilityLabel:l10n_util::GetNSString(IDS_ACCNAME_CANCEL)];
[self navigationItem].leftBarButtonItem = _cancelButton;
// Set up right (pay) button.
@@ -348,12 +355,9 @@ typedef NS_ENUM(NSInteger, ItemType) {
- (void)fillShippingAddressItem:(ShippingAddressItem*)item
withAddress:(autofill::AutofillProfile*)address {
- item.name =
- base::SysUTF16ToNSString(address->GetRawInfo(autofill::NAME_FULL));
- item.address =
- payment_request_utils::AddressLabelFromAutofillProfile(address);
- item.phoneNumber = base::SysUTF16ToNSString(
- address->GetRawInfo(autofill::PHONE_HOME_WHOLE_NUMBER));
+ item.name = NameLabelFromAutofillProfile(address);
+ item.address = AddressLabelFromAutofillProfile(address);
+ item.phoneNumber = PhoneNumberLabelFromAutofillProfile(address);
}
- (void)fillShippingOptionItem:(CollectionViewTextItem*)item

Powered by Google App Engine
This is Rietveld 408576698