requestAutocomplete: Fill |form_structure_| from Online Wallet data (including

chrome/browser/ui/autofill/autofill_dialog_controller_impl.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 "chrome/browser/ui/autofill/autofill_dialog_controller_impl.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 147 matching lines @@
 void GetBillingInfoFromOutputs(const DetailOutputMap& output,
                                CreditCard* card,
                                string16* cvc,
                                AutofillProfile* profile) {
   for (DetailOutputMap::const_iterator it = output.begin();
        it != output.end(); ++it) {
     string16 trimmed;
     TrimWhitespace(it->second, TRIM_ALL, &trimmed);
 
     // Special case CVC as CreditCard just swallows it.
-    if (it->first->type == CREDIT_CARD_VERIFICATION_CODE) {
-      *cvc = trimmed;
+    if (cvc && it->first->type == CREDIT_CARD_VERIFICATION_CODE) {

[Evan Stade, 2013/03/13 01:54:10]

shouldn't you put the if (cvc) check inside this block

[Dan Beam, 2013/03/13 03:05:47]

Done.

+      cvc->assign(trimmed);
     } else {
       // Copy the credit card name to |profile| in addition to |card| as
       // wallet::Instrument requires a recipient name for its billing address.
-      if (it->first->type == CREDIT_CARD_NAME)
+      if (profile && it->first->type == CREDIT_CARD_NAME)
         profile->SetRawInfo(NAME_FULL, trimmed);
 
-      if (AutofillType(it->first->type).group() == AutofillType::CREDIT_CARD)
-        card->SetRawInfo(it->first->type, trimmed);
-      else
-        profile->SetRawInfo(it->first->type, trimmed);
+      if (AutofillType(it->first->type).group() == AutofillType::CREDIT_CARD) {
+        if (card)
+          card->SetRawInfo(it->first->type, trimmed);
+      } else {

[Evan Stade, 2013/03/13 01:54:10]

merge else { if into else if

[Dan Beam, 2013/03/13 03:05:47]

Done.

+        if (profile)
+          profile->SetRawInfo(it->first->type, trimmed);
+      }
     }
   }
 }
 
 // Returns the containing window for the given |web_contents|.  The containing
 // window might be a browser window for a Chrome tab, or it might be a shell
 // window for a platform app.
 BaseWindow* GetBaseWindowForWebContents(
     const content::WebContents* web_contents) {
   Browser* browser = chrome::FindBrowserWithWebContents(web_contents);
@@ skipping 1133 matching lines @@
   return true;
 }
 
 void AutofillDialogControllerImpl::FillOutputForSectionWithComparator(
     DialogSection section,
     const InputFieldComparator& compare) {
   if (!SectionIsActive(section))
     return;
 
   if (!IsManuallyEditingSection(section)) {
-    if (section == SECTION_CC_BILLING) {
-      // TODO(dbeam): implement.
+    if (IsPayingWithWallet()) {
+      bool is_billing = section == SECTION_SHIPPING ||
+                        section == SECTION_CC_BILLING;
+      const DetailInputs& inputs = RequestedFieldsForSection(section);
+      for (size_t i = 0; i < form_structure_.field_count(); ++i) {
+        AutofillField* field = form_structure_.field(i);
+        for (size_t j = 0; j < inputs.size(); ++j) {
+          if (compare.Run(inputs[j], *field)) {
+            field->value = full_wallet_->GetInfo(field->type(), is_billing);
+            break;
+          }
+        }
+      }
     } else {
       scoped_ptr<DataModelWrapper> model = CreateWrapper(section);
       // Only fill in data that is associated with this section.
       const DetailInputs& inputs = RequestedFieldsForSection(section);
       model->FillFormStructure(inputs, compare, &form_structure_);
 
       // CVC needs special-casing because the CreditCard class doesn't store
       // or handle them.
       if (section == SECTION_CC)
         SetCvcResult(view_->GetCvc());
     }
   } else {
     // The user manually input data.
     DetailOutputMap output;
     view_->GetUserInput(section, &output);
 
-    if (IsPayingWithWallet()) {
-      // TODO(dbeam): implement.
-    } else {
-      // Save the info as new or edited data and fill it into |form_structure_|.
-      if (section == SECTION_CC) {
-        CreditCard card;
-        FillFormGroupFromOutputs(output, &card);
+    scoped_ptr<CreditCard> card;
+    scoped_ptr<string16> cvc;
+    if (section == SECTION_CC || section == SECTION_CC_BILLING) {
+      card.reset(new CreditCard());
+      cvc.reset(new string16());
+    }
 
-        if (view_->SaveDetailsLocally())
-          GetManager()->SaveImportedCreditCard(card);
+    AutofillProfile profile;
+    GetBillingInfoFromOutputs(output, card.get(), cvc.get(), &profile);
 
-        FillFormStructureForSection(card, 0, section, compare);
+    // TODO(estade): we should probably edit the existing profile in the cases
+    // where the input data is based on an existing profile (user clicked "Edit"
+    // or autofill popup filled in the form).
+    if (card.get()) {
+      if (view_->SaveDetailsLocally())
+        GetManager()->SaveImportedCreditCard(*card);
 
-        // Again, CVC needs special-casing. Fill it in directly from |output|.
-        SetCvcResult(GetValueForType(output, CREDIT_CARD_VERIFICATION_CODE));
-      } else {
-        AutofillProfile profile;
-        FillFormGroupFromOutputs(output, &profile);
+      FillFormStructureForSection(*card, 0, section, compare);
+    }
 
-        // TODO(estade): we should probably edit the existing profile in the
-        // cases where the input data is based on an existing profile (user
-        // clicked "Edit" or autofill popup filled in the form).
-        if (view_->SaveDetailsLocally())
-          GetManager()->SaveImportedProfile(profile);
+    if (cvc.get() && !cvc->empty())
+      SetCvcResult(*cvc);
 
-        FillFormStructureForSection(profile, 0, section, compare);
-      }
-    }
+    if (view_->SaveDetailsLocally())
+      GetManager()->SaveImportedProfile(profile);
+
+    FillFormStructureForSection(profile, 0, section, compare);
   }
 }
 
 void AutofillDialogControllerImpl::FillOutputForSection(DialogSection section) {
   FillOutputForSectionWithComparator(section,
                                      base::Bind(DetailInputMatchesField));
 }
 
 void AutofillDialogControllerImpl::FillFormStructureForSection(
     const FormGroup& form_group,
@@ skipping 241 matching lines @@
   callback_.Run(&form_structure_);
   callback_ = base::Callback<void(const FormStructure*)>();
 
   if (dialog_type_ == DIALOG_TYPE_REQUEST_AUTOCOMPLETE) {
     // This may delete us.
     Hide();
   }
 }
 
 }  // namespace autofill