requestAutocomplete: change comboboxes to dropdown menus

chrome/browser/ui/autofill/autofill_dialog_controller.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_controller.h"
 
 #include "base/logging.h"
 #include "base/string_split.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
@@ skipping 101 matching lines @@
     const FormData& form,
     const GURL& source_url,
     const content::SSLStatus& ssl_status,
     const base::Callback<void(const FormStructure*)>& callback)
     : profile_(Profile::FromBrowserContext(contents->GetBrowserContext())),
       contents_(contents),
       form_structure_(form),
       source_url_(source_url),
       ssl_status_(ssl_status),
       callback_(callback),
+      ALLOW_THIS_IN_INITIALIZER_LIST(suggested_email_(this)),
+      ALLOW_THIS_IN_INITIALIZER_LIST(suggested_cc_(this)),
+      ALLOW_THIS_IN_INITIALIZER_LIST(suggested_billing_(this)),
+      ALLOW_THIS_IN_INITIALIZER_LIST(suggested_shipping_(this)),
       popup_controller_(NULL) {
   // TODO(estade): |this| should observe PersonalDataManager.
   // TODO(estade): remove duplicates from |form|?
 }
 
 AutofillDialogController::~AutofillDialogController() {
   if (popup_controller_)
     popup_controller_->Hide();
 }
 
@@ skipping 53 matching lines @@
   FilterInputs(form_structure_,
                kBillingInputs,
                arraysize(kBillingInputs),
                &requested_billing_fields_);
 
   FilterInputs(form_structure_,
                kShippingInputs,
                arraysize(kShippingInputs),
                &requested_shipping_fields_);
 
-  GenerateComboboxModels();
+  GenerateSuggestionsModels();
 
   // TODO(estade): don't show the dialog if the site didn't specify the right
   // fields. First we must figure out what the "right" fields are.
   view_.reset(AutofillDialogView::Create(this));
   view_->Show();
 }
 
 string16 AutofillDialogController::DialogTitle() const {
   return l10n_util::GetStringUTF16(IDS_AUTOFILL_DIALOG_TITLE);
 }
@@ skipping 107 matching lines @@
       return &cc_exp_month_combobox_model_;
 
     case CREDIT_CARD_EXP_4_DIGIT_YEAR:
       return &cc_exp_year_combobox_model_;
 
     default:
       return NULL;
   }
 }
 
-ui::ComboboxModel* AutofillDialogController::ComboboxModelForSection(
+ui::MenuModel* AutofillDialogController::MenuModelForSection(
     DialogSection section) {
-  return SuggestionsModelForSection(section);
+  return SuggestionsMenuModelForSection(section);
+}
+
+string16 AutofillDialogController::SuggestionTextForSection(
+    DialogSection section) {
+  SuggestionsMenuModel* model = SuggestionsMenuModelForSection(section);
+  std::string item_key = model->GetItemKeyAt(model->checked_item());
+  if (item_key.empty())
+    return string16();
+
+  // TODO(estade): This doesn't display as much info as it should (for example,
+  // it should show full addresses rather than just a summary).
+  if (section == SECTION_CC) {
+    CreditCard* card = GetManager()->GetCreditCardByGUID(item_key);
+    if (card)
+      return card->Label();
+  } else {
+    AutofillProfile* profile = GetManager()->GetProfileByGUID(item_key);
+    if (profile) {
+      const std::string app_locale = AutofillCountry::ApplicationLocale();
+      return section == SECTION_EMAIL ?
+          profile->GetInfo(EMAIL_ADDRESS, app_locale) : profile->Label();
+    }
+  }
+
+  // TODO(estade): The FormGroup was likely deleted while menu was showing. We
+  // should not let this happen.
+  return string16();
 }
 
 void AutofillDialogController::ViewClosed(DialogAction action) {
   if (action == ACTION_SUBMIT) {
     FillOutputForSection(SECTION_EMAIL);
     FillOutputForSection(SECTION_CC);
     FillOutputForSection(SECTION_BILLING);
     if (view_->UseBillingForShipping()) {
       FillOutputForSectionWithComparator(
           SECTION_BILLING,
@@ skipping 85 matching lines @@
 }
 
 void AutofillDialogController::ClearPreviewedForm() {
   // TODO(estade): implement.
 }
 
 void AutofillDialogController::ControllerDestroyed() {
   popup_controller_ = NULL;
 }
 
-void AutofillDialogController::GenerateComboboxModels() {
+void AutofillDialogController::SuggestionItemSelected(
+    const SuggestionsMenuModel& model) {
+  view_->UpdateSection(SectionForSuggestionsMenuModel(model));
+}
+
+void AutofillDialogController::GenerateSuggestionsModels() {
   PersonalDataManager* manager = GetManager();
   const std::vector<CreditCard*>& cards = manager->credit_cards();
   for (size_t i = 0; i < cards.size(); ++i) {
-    suggested_cc_.AddItem(cards[i]->guid(), cards[i]->Label());
+    suggested_cc_.AddKeyedItem(cards[i]->guid(), cards[i]->Label());
   }
-  suggested_cc_.AddItem("", ASCIIToUTF16("Enter new card"));
+  suggested_cc_.AddKeyedItem("", ASCIIToUTF16("Enter new card"));
 
   const std::vector<AutofillProfile*>& profiles = manager->GetProfiles();
   const std::string app_locale = AutofillCountry::ApplicationLocale();
   for (size_t i = 0; i < profiles.size(); ++i) {
     // TODO(estade): deal with variants.
     if (!IsCompleteProfile(*profiles[i]))
       continue;
 
     string16 email = profiles[i]->GetInfo(EMAIL_ADDRESS, app_locale);
     if (!email.empty())
-      suggested_email_.AddItem(profiles[i]->guid(), email);
-    suggested_billing_.AddItem(profiles[i]->guid(), profiles[i]->Label());
-    suggested_shipping_.AddItem(profiles[i]->guid(), profiles[i]->Label());
+      suggested_email_.AddKeyedItem(profiles[i]->guid(), email);
+    suggested_billing_.AddKeyedItem(profiles[i]->guid(), profiles[i]->Label());
+    suggested_shipping_.AddKeyedItem(profiles[i]->guid(), profiles[i]->Label());
   }
-  suggested_billing_.AddItem("", ASCIIToUTF16("Enter new billing"));
-  suggested_email_.AddItem("", ASCIIToUTF16("Enter new email"));
-  suggested_shipping_.AddItem("", ASCIIToUTF16("Enter new shipping"));
+  suggested_billing_.AddKeyedItem("", ASCIIToUTF16("Enter new billing"));
+  suggested_email_.AddKeyedItem("", ASCIIToUTF16("Enter new email"));
+  suggested_shipping_.AddKeyedItem("", ASCIIToUTF16("Enter new shipping"));
 }
 
-
 bool AutofillDialogController::IsCompleteProfile(
     const AutofillProfile& profile) {
   const std::string app_locale = AutofillCountry::ApplicationLocale();
   for (size_t i = 0; i < requested_shipping_fields_.size(); ++i) {
     if (profile.GetInfo(requested_shipping_fields_[i].type,
                         app_locale).empty()) {
       return false;
     }
   }
 
   return true;
 }
 
 void AutofillDialogController::FillOutputForSectionWithComparator(
     DialogSection section, const InputFieldComparator& compare) {
-  int suggestion_selection = view_->GetSuggestionSelection(section);
-  SuggestionsComboboxModel* model = SuggestionsModelForSection(section);
+  SuggestionsMenuModel* model = SuggestionsMenuModelForSection(section);
+  std::string guid = model->GetItemKeyAt(model->checked_item());
   PersonalDataManager* manager = GetManager();
-  if (suggestion_selection < model->GetItemCount() - 1) {
-    std::string guid = model->GetItemKeyAt(suggestion_selection);
+  if (!guid.empty()) {
     FormGroup* form_group = section == SECTION_CC ?
         static_cast<FormGroup*>(manager->GetCreditCardByGUID(guid)) :
         static_cast<FormGroup*>(manager->GetProfileByGUID(guid));
     // TODO(estade): we shouldn't let this happen.
     if (!form_group)
       return;
 
     FillFormStructureForSection(*form_group, section, compare);
   } else {
     // The user manually input data.
@@ skipping 46 matching lines @@
     const DetailInputs& inputs = RequestedFieldsForSection(section);
     for (size_t j = 0; j < inputs.size(); ++j) {
       if (compare.Run(inputs[j], *field)) {
         form_group.FillFormField(*field, 0, field);
         break;
       }
     }
   }
 }
 
-SuggestionsComboboxModel* AutofillDialogController::SuggestionsModelForSection(
+SuggestionsMenuModel* AutofillDialogController::SuggestionsMenuModelForSection(
     DialogSection section) {
   switch (section) {
     case SECTION_EMAIL:
       return &suggested_email_;
     case SECTION_CC:
       return &suggested_cc_;
     case SECTION_BILLING:
       return &suggested_billing_;
     case SECTION_SHIPPING:
       return &suggested_shipping_;
   }
 
   NOTREACHED();
   return NULL;
 }
 
+DialogSection AutofillDialogController::SectionForSuggestionsMenuModel(
+    const SuggestionsMenuModel& model) {
+  if (&model == &suggested_email_)
+    return SECTION_EMAIL;
+
+  if (&model == &suggested_cc_)
+    return SECTION_CC;
+
+  if (&model == &suggested_billing_)
+    return SECTION_BILLING;
+
+  DCHECK_EQ(&model, &suggested_shipping_);
+  return SECTION_SHIPPING;
+}
+
 PersonalDataManager* AutofillDialogController::GetManager() {
   return PersonalDataManagerFactory::GetForProfile(profile_);
 }
 
 }  // namespace autofill