Revamp Autofill Wallet settings

components/autofill/core/browser/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/core/browser/personal_data_manager.h"
 
 #include <algorithm>
 #include <functional>
 #include <iterator>
 
@@ skipping 647 matching lines @@
   if (existing_credit_card->record_type() == CreditCard::MASKED_SERVER_CARD) {
     database_->UnmaskServerCreditCard(credit_card.server_id(),
                                       credit_card.number());
   } else {
     database_->MaskServerCreditCard(credit_card.server_id());
   }
 
   Refresh();
 }
 
+void PersonalDataManager::ResetFullServerCard(const std::string& guid) {
+  for (const CreditCard* card : server_credit_cards_) {
+    if (card->guid() == guid) {
+      DCHECK_EQ(card->record_type(), CreditCard::FULL_SERVER_CARD);
+      CreditCard card_copy = *card;
+      card_copy.set_record_type(CreditCard::MASKED_SERVER_CARD);
+      card_copy.SetNumber(card->LastFourDigits());
+      UpdateServerCreditCard(card_copy);
+      break;

[please use gerrit instead, 2015/03/12 20:40:12]

Nit: "return" here and "NOTREACHED()" after the loop.

[Evan Stade, 2015/03/12 21:28:03]

the general pattern in PDM is to fail silently if the requested item isn't
present. I think it's because the UI can lag slightly behind the backend (due to
race conditions).

+    }
+  }
+}
+
 void PersonalDataManager::ResetFullServerCards() {
   for (const CreditCard* card : server_credit_cards_) {
     CreditCard card_copy = *card;
     if (card_copy.record_type() == CreditCard::FULL_SERVER_CARD) {
       card_copy.set_record_type(CreditCard::MASKED_SERVER_CARD);
       card_copy.SetNumber(card->LastFourDigits());
       UpdateServerCreditCard(card_copy);
     }
   }
 }
@@ skipping 659 matching lines @@
   }
   if (IsExperimentalWalletIntegrationEnabled() &&
       pref_service_->GetBoolean(prefs::kAutofillWalletImportEnabled)) {
     profiles_.insert(
         profiles_.end(), server_profiles_.begin(), server_profiles_.end());
   }
   return profiles_;
 }
 
 }  // namespace autofill