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

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

Issue 2721553004: Remove auto raw pointer deduction from non-linux specific code. (Closed)
Patch Set: Created 3 years, 10 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.mm
diff --git a/ios/chrome/browser/payments/payment_request.mm b/ios/chrome/browser/payments/payment_request.mm
index e61c810571beced6a74c25b99a209b188869f545..dce9306f7b8d8e3f1211e44337c3d389b7ffcadf 100644
--- a/ios/chrome/browser/payments/payment_request.mm
+++ b/ios/chrome/browser/payments/payment_request.mm
@@ -53,7 +53,7 @@ payments::CurrencyFormatter* PaymentRequest::GetOrCreateCurrencyFormatter() {
}
void PaymentRequest::PopulateProfileCache() {
- for (const auto& profile : personal_data_manager_->GetProfilesToSuggest()) {
+ for (auto* profile : personal_data_manager_->GetProfilesToSuggest()) {
profile_cache_.push_back(
base::MakeUnique<autofill::AutofillProfile>(*profile));
profiles_.push_back(profile_cache_.back().get());
@@ -74,7 +74,7 @@ void PaymentRequest::PopulateCreditCardCache() {
std::vector<autofill::CreditCard*> credit_cards =
personal_data_manager_->GetCreditCardsToSuggest();
- for (const auto& credit_card : credit_cards) {
+ for (auto* credit_card : credit_cards) {
std::string spec_card_type =
autofill::data_util::GetPaymentRequestData(credit_card->type())
.basic_card_payment_type;
@@ -101,7 +101,7 @@ void PaymentRequest::PopulateShippingOptionCache() {
[](web::PaymentShippingOption& option) { return &option; });
selected_shipping_option_ = nullptr;
- for (const auto& shipping_option : shipping_options_) {
+ for (auto* shipping_option : shipping_options_) {
if (shipping_option->selected) {
// If more than one option has |selected| set, the last one in the
// sequence should be treated as the selected item.

Powered by Google App Engine
This is Rietveld 408576698