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

Unified Diff: components/autofill/core/browser/personal_data_manager.h

Issue 2033323003: Revert of [Autofill] Sort profiles and credit cards by frecency in PaymentRequest. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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/autofill/core/browser/personal_data_manager.h
diff --git a/components/autofill/core/browser/personal_data_manager.h b/components/autofill/core/browser/personal_data_manager.h
index 2f2688b3d8a40fce2e4db7bbc9b37cd671a784a0..a62fb6049f68ca5584b30422fde324924ce79490 100644
--- a/components/autofill/core/browser/personal_data_manager.h
+++ b/components/autofill/core/browser/personal_data_manager.h
@@ -180,9 +180,6 @@
// Returns true if there is some data synced from Wallet.
bool HasServerData() const;
- // Returns the profiles to suggest to the user, ordered by frecency.
- const std::vector<AutofillProfile*> GetProfilesToSuggest() const;
-
// Loads profiles that can suggest data for |type|. |field_contents| is the
// part the user has already typed. |field_is_autofilled| is true if the field
// has already been autofilled. |other_field_types| represents the rest of
@@ -193,11 +190,6 @@
bool field_is_autofilled,
const std::vector<ServerFieldType>& other_field_types);
- // Returns the credit cards to suggest to the user. Those have been deduped
- // and ordered by frecency with the expired cards put at the end of the
- // vector.
- const std::vector<CreditCard*> GetCreditCardsToSuggest() const;
-
// Gets credit cards that can suggest data for |type|. See
// GetProfileSuggestions for argument descriptions. The variant in each
// GUID pair should be ignored.
@@ -246,7 +238,7 @@
// local duplicates, and local cards are preferred over their masked server
// card duplicate.
static void DedupeCreditCardToSuggest(
- std::list<CreditCard*>* cards_to_suggest);
+ std::list<const CreditCard*>* cards_to_suggest);
protected:
// Only PersonalDataManagerFactory and certain tests can create instances of
@@ -406,7 +398,7 @@
std::vector<Suggestion> GetSuggestionsForCards(
const AutofillType& type,
const base::string16& field_contents,
- const std::vector<CreditCard*>& cards_to_suggest) const;
+ const std::list<const CreditCard*>& cards_to_suggest) const;
const std::string app_locale_;

Powered by Google App Engine
This is Rietveld 408576698