Chromium Code Reviews| Index: components/autofill/android/java/src/org/chromium/components/autofill/AutofillPopup.java |
| diff --git a/components/autofill/android/java/src/org/chromium/components/autofill/AutofillPopup.java b/components/autofill/android/java/src/org/chromium/components/autofill/AutofillPopup.java |
| index 3a5bd066ed320f6931599a82835818eb984342f9..df6f4299a11fcb4598e6b10a36d2d174842047d4 100644 |
| --- a/components/autofill/android/java/src/org/chromium/components/autofill/AutofillPopup.java |
| +++ b/components/autofill/android/java/src/org/chromium/components/autofill/AutofillPopup.java |
| @@ -67,10 +67,12 @@ public class AutofillPopup extends DropdownPopupWindow implements AdapterView.On |
| * suggestion label's type face is {@code Typeface.NORMAL}. |
| * @param dropdownItemHeight height of each dropdown item in dimension independent pixel units, |
| * 0 if unspecified. |
| + * @param margin Margin for icon, label and between icon and label in dimension independent |
| + * pixel units, 0 if not specified in experiment. |
|
Mathieu
2017/01/27 00:40:43
nit: remove "in experiment"
csashi
2017/01/27 01:21:31
Done.
|
| */ |
| @SuppressLint("InlinedApi") |
| public void filterAndShow(AutofillSuggestion[] suggestions, boolean isRtl, |
| - int backgroundColor, int dividerColor, int dropdownItemHeight) { |
| + int backgroundColor, int dividerColor, int dropdownItemHeight, int margin) { |
| mSuggestions = new ArrayList<AutofillSuggestion>(Arrays.asList(suggestions)); |
| // Remove the AutofillSuggestions with IDs that are not supported by Android |
| ArrayList<DropdownItem> cleanedData = new ArrayList<DropdownItem>(); |
| @@ -87,7 +89,8 @@ public class AutofillPopup extends DropdownPopupWindow implements AdapterView.On |
| setAdapter(new DropdownAdapter(mContext, cleanedData, separators, |
| backgroundColor == Color.TRANSPARENT ? null : backgroundColor, |
| dividerColor == Color.TRANSPARENT ? null : dividerColor, |
| - dropdownItemHeight == 0 ? null : dropdownItemHeight)); |
| + dropdownItemHeight == 0 ? null : dropdownItemHeight, |
| + margin == 0 ? null : margin)); |
| setRtl(isRtl); |
| show(); |
| getListView().setOnItemLongClickListener(this); |