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

Unified Diff: components/payments/content/payment_request_state.cc

Issue 2884393002: [WebPayments] Adding FilterProfilesForShipping to profile comparator (Closed)
Patch Set: Created 3 years, 7 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: components/payments/content/payment_request_state.cc
diff --git a/components/payments/content/payment_request_state.cc b/components/payments/content/payment_request_state.cc
index fb728d12cfe2ffbed3dd485c9390ad42b32e0836..a09370d8a72b4adfcdd05aacfa472acc3ba5e1ed 100644
--- a/components/payments/content/payment_request_state.cc
+++ b/components/payments/content/payment_request_state.cc
@@ -228,8 +228,6 @@ void PaymentRequestState::PopulateProfileCache() {
for (size_t i = 0; i < profiles.size(); i++) {
profile_cache_.push_back(
base::MakeUnique<autofill::AutofillProfile>(*profiles[i]));
-
- shipping_profiles_.push_back(profile_cache_[i].get());
}
std::vector<autofill::AutofillProfile*> raw_profiles_for_filtering(
@@ -242,6 +240,8 @@ void PaymentRequestState::PopulateProfileCache() {
contact_profiles_ = profile_comparator()->FilterProfilesForContact(
raw_profiles_for_filtering);
+ shipping_profiles_ = profile_comparator()->FilterProfilesForShipping(
Mathieu 2017/05/16 20:16:51 can you also call this in payment_request.mm (http
tmartino 2017/05/18 20:49:36 Done
+ raw_profiles_for_filtering);
// Create the list of available instruments. A copy of each card will be made
// by their respective AutofillPaymentInstrument.

Powered by Google App Engine
This is Rietveld 408576698