[Autofill] Suggest expired credit cards last.

components/autofill/core/browser/personal_data_manager.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_
 #define COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_
 
 #include <memory>
 #include <set>
 #include <vector>
@@ skipping 372 matching lines @@
   bool ImportCreditCard(const FormStructure& form,
                         bool should_return_local_card,
                         std::unique_ptr<CreditCard>* imported_credit_card);
 
   // Functionally equivalent to GetProfiles(), but also records metrics if
   // |record_metrics| is true. Metrics should be recorded when the returned
   // profiles will be used to populate the fields shown in an Autofill popup.
   const std::vector<AutofillProfile*>& GetProfiles(
       bool record_metrics) const;
 
-  // Fills |cards_to_suggest| with valid credit cards to suggest based on the
-  // |type| and |field_contents| of the credit card field. The cards are ordered
-  // by frecency.
-  void GetOrderedCardsToSuggest(
+  // Returns credit card suggestions based on the |cards_to_suggest| and the
+  // |type| and |field_contents| of the credit card field.
+  std::vector<Suggestion> GetSuggestionsForCards(
       const AutofillType& type,
       const base::string16& field_contents,
-      std::list<const CreditCard*>* cards_to_suggest) const;
-
-  // Returns the suggestions to display for the |cards_to_suggest| based on the
-  // |type| of the credit card field.
-  std::vector<Suggestion> GetSuggestionsForCards(
-      const std::list<const CreditCard*>& cards_to_suggest,
-      const AutofillType& type) const;
+      const std::list<const CreditCard*>& cards_to_suggest) const;
 
   const std::string app_locale_;
 
   // The default country code for new addresses.
   mutable std::string default_country_code_;
 
   // The PrefService that this instance uses. Must outlive this instance.
   PrefService* pref_service_;
 
   // The AccountTrackerService that this instance uses. Must outlive this
@@ skipping 19 matching lines @@
 
   // An observer to listen for changes to prefs::kAutofillWalletImportEnabled.
   std::unique_ptr<BooleanPrefMember> wallet_enabled_pref_;
 
   DISALLOW_COPY_AND_ASSIGN(PersonalDataManager);
 };
 
 }  // namespace autofill
 
 #endif  // COMPONENTS_AUTOFILL_CORE_BROWSER_PERSONAL_DATA_MANAGER_H_