Index: components/autofill/core/browser/personal_data_manager.cc |
diff --git a/components/autofill/core/browser/personal_data_manager.cc b/components/autofill/core/browser/personal_data_manager.cc |
index ca41fd5b5cf8a22f8287fdc02fb3bdc74b62855d..fd4b4933e5d8682ae6eafb03be19422a51d7ce70 100644 |
--- a/components/autofill/core/browser/personal_data_manager.cc |
+++ b/components/autofill/core/browser/personal_data_manager.cc |
@@ -30,6 +30,7 @@ |
#include "components/autofill/core/browser/validation.h" |
#include "components/autofill/core/common/autofill_pref_names.h" |
#include "components/autofill/core/common/autofill_switches.h" |
+#include "components/autofill/core/common/autofill_util.h" |
#include "components/signin/core/browser/account_tracker_service.h" |
#include "components/signin/core/common/signin_pref_names.h" |
#include "third_party/libaddressinput/src/cpp/include/libaddressinput/address_data.h" |
@@ -795,14 +796,27 @@ std::vector<Suggestion> PersonalDataManager::GetProfileSuggestions( |
continue; |
base::string16 value_canon = |
AutofillProfile::CanonicalizeProfileString(value); |
- if (base::StartsWith(value_canon, field_contents_canon, true)) { |
- // Prefix match, add suggestion. |
+ bool prefix_matched_suggestion = |
+ base::StartsWith(value_canon, field_contents_canon, true); |
+ if (prefix_matched_suggestion || |
+ ContainsTokenThatStartsWith(value, field_contents, false)) { |
matched_profiles.push_back(profile); |
suggestions.push_back(Suggestion(value)); |
suggestions.back().backend_id = profile->guid(); |
+ suggestions.back().match = prefix_matched_suggestion |
+ ? Suggestion::PREFIX_MATCH |
+ : Suggestion::SUBSTRING_MATCH; |
} |
} |
+ // Prefix matches should precede other token matches. |
+ if (IsFeatureSubstringMatchEnabled()) { |
+ std::stable_sort(suggestions.begin(), suggestions.end(), |
+ [](const Suggestion& a, const Suggestion& b) { |
+ return a.match < b.match; |
+ }); |
+ } |
+ |
// Don't show two suggestions if one is a subset of the other. |
std::vector<AutofillProfile*> unique_matched_profiles; |
std::vector<Suggestion> unique_suggestions; |
@@ -854,6 +868,7 @@ std::vector<Suggestion> PersonalDataManager::GetCreditCardSuggestions( |
return std::vector<Suggestion>(); |
std::list<const CreditCard*> cards_to_suggest; |
+ std::list<const CreditCard*> substring_matched_cards; |
for (const CreditCard* credit_card : GetCreditCards()) { |
// The value of the stored data for this field type in the |credit_card|. |
base::string16 creditcard_field_value = |
@@ -872,12 +887,24 @@ std::vector<Suggestion> PersonalDataManager::GetCreditCardSuggestions( |
field_contents.size() >= 6)) { |
continue; |
} |
- } else if (!base::StartsWith(creditcard_field_value, field_contents, |
- false)) { |
- continue; |
+ cards_to_suggest.push_back(credit_card); |
+ } else if (base::StartsWith(creditcard_field_value, field_contents, |
+ false)) { |
+ cards_to_suggest.push_back(credit_card); |
+ } else if (ContainsTokenThatStartsWith(creditcard_field_value, |
+ field_contents, false)) { |
+ substring_matched_cards.push_back(credit_card); |
} |
+ } |
- cards_to_suggest.push_back(credit_card); |
+ cards_to_suggest.sort(RankByMfu); |
+ |
+ // Prefix matches should precede other token matches. |
+ if (IsFeatureSubstringMatchEnabled()) { |
+ substring_matched_cards.sort(RankByMfu); |
+ cards_to_suggest.insert(cards_to_suggest.end(), |
+ substring_matched_cards.begin(), |
+ substring_matched_cards.end()); |
} |
// De-dupe card suggestions. Full server cards shadow local cards, and |
@@ -905,8 +932,6 @@ std::vector<Suggestion> PersonalDataManager::GetCreditCardSuggestions( |
} |
} |
- cards_to_suggest.sort(RankByMfu); |
- |
std::vector<Suggestion> suggestions; |
for (const CreditCard* credit_card : cards_to_suggest) { |
// Make a new suggestion. |