Change the behavior of the [X] Save details to Wallet checkbox notification to:

chrome/browser/ui/autofill/autofill_dialog_models.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/autofill/autofill_dialog_models.h"
 
 #include "base/bind.h"
 #include "base/prefs/pref_service.h"
 #include "base/stringprintf.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 132 matching lines @@
 bool AccountChooserModel::GetAcceleratorForCommandId(
     int command_id,
     ui::Accelerator* accelerator) {
   return false;
 }
 
 void AccountChooserModel::ExecuteCommand(int command_id, int event_flags) {
   if (checked_item_ == command_id)
     return;
 
+  prefs_->SetBoolean(prefs::kAutofillDialogPayWithoutWallet,

[Dan Beam, 2013/04/05 02:45:57]

should this model know about prefs?

[aruslan, 2013/04/05 03:45:35]

I'd rather didn't pretend the account chooser knows more than the set of
accounts and the user selection. In this case all this wiring should go to the
controller and everyone is happy.

+                     command_id == kAutofillItemId);
+
   checked_item_ = command_id;
   account_delegate_->AccountChoiceChanged();
 }
 
 void AccountChooserModel::SetHadWalletError() {
   had_wallet_error_ = true;
-  checked_item_ = kAutofillItemId;
-  account_delegate_->AccountChoiceChanged();
+  ExecuteCommand(kAutofillItemId, 0);
 }
 
 void AccountChooserModel::SetHadWalletSigninError() {
-  checked_item_ = kAutofillItemId;
-  account_delegate_->AccountChoiceChanged();
+  ExecuteCommand(kAutofillItemId, 0);
 }
 
 bool AccountChooserModel::WalletIsSelected() const {
   return checked_item_ == kWalletItemId;
 }
 
 void AccountChooserModel::PrefChanged(const std::string& pref) {
-  DCHECK(pref == prefs::kAutofillDialogPayWithoutWallet);
+  DCHECK_EQ(pref, prefs::kAutofillDialogPayWithoutWallet);
   UpdateCheckmarkFromPref();
   account_delegate_->AccountChoiceChanged();
 }
 
 void AccountChooserModel::UpdateCheckmarkFromPref() {
   if (prefs_->GetBoolean(prefs::kAutofillDialogPayWithoutWallet))

[Dan Beam, 2013/04/05 02:45:57]

^ it already did here, but this could be changed

     checked_item_ = kAutofillItemId;
   else
     checked_item_ = kWalletItemId;
 }
 
 // MonthComboboxModel ----------------------------------------------------------
 
 MonthComboboxModel::MonthComboboxModel() {}
 
 MonthComboboxModel::~MonthComboboxModel() {}
@@ skipping 29 matching lines @@
 }
 
 string16 YearComboboxModel::GetItemAt(int index) {
   if (index == 0)
     return string16();
 
   return base::IntToString16(this_year_ + index - 1);
 }
 
 }  // autofill