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

Unified Diff: chrome/browser/ui/android/autofill/autofill_popup_view_android.h

Issue 11636040: AutofillPopupController clarifications + simplifications. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: new workstation 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/ui/android/autofill/autofill_popup_view_android.h
diff --git a/chrome/browser/ui/android/autofill/autofill_popup_view_android.h b/chrome/browser/ui/android/autofill/autofill_popup_view_android.h
index 9cf4b900727aa291a90f544a636fb4572d8199d0..76825192cba3024321815e6a6d954a1d9b0c0549 100644
--- a/chrome/browser/ui/android/autofill/autofill_popup_view_android.h
+++ b/chrome/browser/ui/android/autofill/autofill_popup_view_android.h
@@ -24,11 +24,12 @@ class AutofillPopupViewAndroid : public AutofillPopupView {
// --------------------------------------------------------------------------
// Methods called from Java via JNI
// --------------------------------------------------------------------------
- // Called when an autofill item was selected.
+ // Called when an autofill item was selected. TODO(estade): remove |name| and
+ // |unique_id|.
void SuggestionSelected(JNIEnv* env,
jobject obj,
jint list_index,
- jstring value,
+ jstring name,
jint unique_id);
void Dismissed(JNIEnv *env, jobject obj);

Powered by Google App Engine
This is Rietveld 408576698