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

Unified Diff: ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java

Issue 168283003: Android: pass AutofillSuggestion to the logger. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 months 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
« no previous file with comments | « no previous file | ui/android/java/src/org/chromium/ui/autofill/AutofillSuggestion.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java
diff --git a/ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java b/ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java
index 06126a3d6a2f45175cade86fefe7c28ee315c45c..991fc70a9fc634422270dab201c5eb2c47e813e3 100644
--- a/ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java
+++ b/ui/android/java/src/org/chromium/ui/autofill/AutofillPopup.java
@@ -60,7 +60,7 @@ public class AutofillPopup extends ListPopupWindow implements AdapterView.OnItem
* the autofill.
*/
public interface AutofillLogger {
- public void logSuggestionSelected(String fieldName);
+ public void logSuggestionSelected(AutofillSuggestion autofillSuggestion);
}
private static AutofillLogger sAutofillLogger = null;
@@ -237,7 +237,7 @@ public class AutofillPopup extends ListPopupWindow implements AdapterView.OnItem
int listIndex = mSuggestions.indexOf(selectedSuggestion);
assert listIndex > -1;
if (sAutofillLogger != null) {
- sAutofillLogger.logSuggestionSelected(selectedSuggestion.mLabel);
+ sAutofillLogger.logSuggestionSelected(selectedSuggestion);
}
mAutofillCallback.suggestionSelected(listIndex);
}
« no previous file with comments | « no previous file | ui/android/java/src/org/chromium/ui/autofill/AutofillSuggestion.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698