AutofillPopupController clarifications + simplifications.

chrome/browser/autofill/autofill_external_delegate.cc

Index: chrome/browser/autofill/autofill_external_delegate.cc
diff --git a/chrome/browser/autofill/autofill_external_delegate.cc b/chrome/browser/autofill/autofill_external_delegate.cc
index c58ff06f4634fa71c58143a13a12b06eb253064e..155089492edb4dc407793b09805637a88e95917d 100644
--- a/chrome/browser/autofill/autofill_external_delegate.cc
+++ b/chrome/browser/autofill/autofill_external_delegate.cc
@@ -223,14 +223,9 @@ void AutofillExternalDelegate::DidEndTextFieldEditing() {
   has_shown_autofill_popup_for_current_edit_ = false;
 }
 
-bool AutofillExternalDelegate::DidAcceptAutofillSuggestion(
+void AutofillExternalDelegate::DidAcceptAutofillSuggestion(
     const string16& value,
-    int unique_id,
-    unsigned index) {
-  // If the selected element is a warning we don't want to do anything.
-  if (unique_id == WebAutofillClient::MenuItemIDWarningMessage)
-    return false;
-
+    int unique_id) {
   RenderViewHost* host = web_contents_->GetRenderViewHost();
 
   if (unique_id == WebAutofillClient::MenuItemIDAutofillOptions) {
@@ -257,8 +252,6 @@ bool AutofillExternalDelegate::DidAcceptAutofillSuggestion(
   }
 
   HideAutofillPopup();
-
-  return true;
 }
 
 void AutofillExternalDelegate::ClearPreviewedForm() {