Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(477)

Unified Diff: chrome/browser/autofill/autofill_external_delegate.h

Issue 11636040: AutofillPopupController clarifications + simplifications. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: relative patchset Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/autofill/autofill_external_delegate.h
diff --git a/chrome/browser/autofill/autofill_external_delegate.h b/chrome/browser/autofill/autofill_external_delegate.h
index 04dd109b547c8fdc2b74eb7ea6af126031167afc..258d8a41999592ad0c283708bb913d4e17afbbe3 100644
--- a/chrome/browser/autofill/autofill_external_delegate.h
+++ b/chrome/browser/autofill/autofill_external_delegate.h
@@ -48,9 +48,8 @@ class AutofillExternalDelegate
// AutofillPopupDelegate implementation.
virtual void SelectAutofillSuggestion(int unique_id) OVERRIDE;
- virtual bool DidAcceptAutofillSuggestion(const string16& value,
- int unique_id,
- unsigned index) OVERRIDE;
+ virtual void DidAcceptAutofillSuggestion(const string16& value,
+ int unique_id) OVERRIDE;
Ilya Sherman 2012/12/20 04:34:52 nit: Should we drop "Autofill" from these method n
Evan Stade 2012/12/20 20:01:25 Done.
virtual void ClearPreviewedForm() OVERRIDE;
virtual void RemoveAutocompleteEntry(const string16& value) OVERRIDE;
virtual void RemoveAutofillProfileOrCreditCard(int unique_id) OVERRIDE;

Powered by Google App Engine
This is Rietveld 408576698