AutofillPopupController clarifications + simplifications.

chrome/browser/ui/autofill/autofill_dialog_controller.cc

Index: chrome/browser/ui/autofill/autofill_dialog_controller.cc
diff --git a/chrome/browser/ui/autofill/autofill_dialog_controller.cc b/chrome/browser/ui/autofill/autofill_dialog_controller.cc
index ecb1335f90631541f7b535234f600a700f78db27..04d8b0a2ebb6a369be594e3b3be82f789f813db3 100644
--- a/chrome/browser/ui/autofill/autofill_dialog_controller.cc
+++ b/chrome/browser/ui/autofill/autofill_dialog_controller.cc
@@ -400,16 +400,15 @@ void AutofillDialogController::SelectAutofillSuggestion(int unique_id) {
   // TODO(estade): implement.
 }
 
-bool AutofillDialogController::DidAcceptAutofillSuggestion(
+void AutofillDialogController::DidAcceptAutofillSuggestion(
     const string16& value,
-    int unique_id,
-    unsigned index) {
+    int unique_id) {
   const PersonalDataManager::GUIDPair& pair = popup_guids_[unique_id];
   // TODO(estade): need to use the variant, |pair.second|.
   AutofillProfile* profile = GetManager()->GetProfileByGUID(pair.first);
   // TODO(estade): we shouldn't let this happen.
   if (!profile)
-    return false;
+    return;
 
   // TODO(estade): implement for all sections.
   FillInputFromFormGroup(profile, &requested_billing_fields_);
@@ -418,7 +417,6 @@ bool AutofillDialogController::DidAcceptAutofillSuggestion(
   // TODO(estade): not sure why it's necessary to do this explicitly.
   popup_controller_->Hide();
   ControllerDestroyed();
-  return true;
 }
 
 void AutofillDialogController::RemoveAutocompleteEntry(const string16& value) {