Make PersonalDataManager use GetCreditCards() (rather than using credit_cards_ directly) so Autofill

components/autofill/browser/test_personal_data_manager.cc

 // 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.
 
 #include "components/autofill/browser/test_personal_data_manager.h"
 
 #include "components/autofill/browser/personal_data_manager_observer.h"
 
 namespace autofill {
 
@@ skipping 11 matching lines @@
 void TestPersonalDataManager::AddTestingCreditCard(CreditCard* credit_card) {
   credit_cards_.push_back(credit_card);
   FOR_EACH_OBSERVER(PersonalDataManagerObserver, observers_,
                     OnPersonalDataChanged());
 }
 
 const std::vector<AutofillProfile*>& TestPersonalDataManager::GetProfiles() {
   return profiles_;
 }
 
-const std::vector<CreditCard*>& TestPersonalDataManager::credit_cards() const {
+const std::vector<CreditCard*>& TestPersonalDataManager::

[Lei Zhang, 2013/05/01 04:08:52]

nit: I prefer this style, but if you've been doing this consistently, then ok.

const std::vector<CreditCard*>&
TestPersonalDataManager::GetCreditCards() const {

[Dan Beam, 2013/05/01 04:19:36]

yeah, estade@ has been doing this consistently (that I've seen), yes

+    GetCreditCards() const {
   return credit_cards_;
 }
 
 void TestPersonalDataManager::SaveImportedProfile(
     const AutofillProfile& imported_profile) {
   imported_profile_ = imported_profile;
 }
 
 }  // namespace autofill