AutofillPopupController clarifications + simplifications.

chrome/browser/ui/autofill/autofill_popup_delegate.h

Index: chrome/browser/ui/autofill/autofill_popup_delegate.h
diff --git a/chrome/browser/ui/autofill/autofill_popup_delegate.h b/chrome/browser/ui/autofill/autofill_popup_delegate.h
index 38f99e3a8efea95038291dac91c3a0288dffd1be..d7e6d3ef2e83672cb0bda342e43941bcccae5d63 100644
--- a/chrome/browser/ui/autofill/autofill_popup_delegate.h
+++ b/chrome/browser/ui/autofill/autofill_popup_delegate.h
@@ -16,12 +16,12 @@ class AutofillPopupDelegate {
   virtual void SelectAutofillSuggestion(int unique_id) = 0;
 
   // Inform the delegate that a row in the popup has been chosen.
-  // TODO(estade): does this really need to return a value?
-  virtual bool DidAcceptAutofillSuggestion(const string16& value,
-                                           int unique_id,
-                                           unsigned index) = 0;
+  // TODO(estade): this should only pass |unique_id|.

[Ilya Sherman, 2012/12/20 04:34:52]

All autocomplete items have the same unique id, which is 0.  Ditto for passwords
and datalist elements.

[Evan Stade, 2012/12/20 20:01:25]

doesn't sound very unique then. Those things should probably be give actually
unique identifiers.

[Ilya Sherman, 2012/12/20 21:57:42]

Everything other than Autofill profiles and credit cards is uniquely identified
by its value.  The profiles and credit cards, OTOH, need more.  What we should
really do is to split this into two fields: a required one that specifies the
data type, and an optional (i.e. sometimes NULL) one that specifies which
FormGroup the data comes from.  I think that will be easier to do once the old
UI is fully dead.

[Evan Stade, 2012/12/20 23:03:59]

how would the delegate know which password you're accepting?  

I've removed the TODO.

[Ilya Sherman, 2012/12/20 23:11:55]

A given username on a given site only ever has one password associated with it.

+  virtual void DidAcceptAutofillSuggestion(const string16& value,
+                                           int unique_id) = 0;
 
   // Remove the given Autocomplete entry from the DB.
+  // TODO(estade): this should pass |unique_id|.
   virtual void RemoveAutocompleteEntry(const string16& value) = 0;
 
   // Remove the given Autofill profile or credit credit.