| Index: chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc
|
| diff --git a/chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc b/chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc
|
| index 006fffbae9ed80d33047cfd448e6e3cfe7c75580..867c0d6edf500583077e0d974325ff1d0ea975d1 100644
|
| --- a/chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc
|
| +++ b/chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc
|
| @@ -24,16 +24,14 @@ namespace autofill {
|
| namespace {
|
|
|
| void AddToJavaArray(const Suggestion& suggestion,
|
| - const AutofillPopupController& controller,
|
| + int icon_id,
|
| JNIEnv* env,
|
| jobjectArray data_array,
|
| size_t position,
|
| bool deletable) {
|
| int android_icon_id = 0;
|
| - if (!suggestion.icon.empty()) {
|
| - android_icon_id = ResourceMapper::MapFromChromiumId(
|
| - controller.GetIconResourceID(suggestion.icon));
|
| - }
|
| + if (!suggestion.icon.empty())
|
| + android_icon_id = ResourceMapper::MapFromChromiumId(icon_id);
|
|
|
| Java_AutofillKeyboardAccessoryBridge_addToAutofillSuggestionArray(
|
| env, data_array, position,
|
| @@ -46,7 +44,8 @@ void AddToJavaArray(const Suggestion& suggestion,
|
|
|
| AutofillKeyboardAccessoryView::AutofillKeyboardAccessoryView(
|
| AutofillPopupController* controller)
|
| - : controller_(controller),
|
| + : AutofillPopupView(controller),
|
| + controller_(controller),
|
| deleting_index_(-1) {
|
| JNIEnv* env = base::android::AttachCurrentThread();
|
| java_object_.Reset(Java_AutofillKeyboardAccessoryBridge_create(env));
|
| @@ -89,8 +88,8 @@ void AutofillKeyboardAccessoryView::UpdateBoundsAndRedrawPopup() {
|
| for (size_t i = 0; i < count; ++i) {
|
| const Suggestion& suggestion = controller_->GetSuggestionAt(i);
|
| if (suggestion.frontend_id == POPUP_ITEM_ID_CLEAR_FORM) {
|
| - AddToJavaArray(suggestion, *controller_, env, data_array.obj(), position,
|
| - false);
|
| + AddToJavaArray(suggestion, GetIconResourceID(suggestion.icon), env,
|
| + data_array.obj(), position, false);
|
| positions_[position++] = i;
|
| }
|
| }
|
| @@ -100,8 +99,8 @@ void AutofillKeyboardAccessoryView::UpdateBoundsAndRedrawPopup() {
|
| if (suggestion.frontend_id != POPUP_ITEM_ID_CLEAR_FORM) {
|
| bool deletable =
|
| controller_->GetRemovalConfirmationText(i, nullptr, nullptr);
|
| - AddToJavaArray(suggestion, *controller_, env, data_array.obj(), position,
|
| - deletable);
|
| + AddToJavaArray(suggestion, GetIconResourceID(suggestion.icon), env,
|
| + data_array.obj(), position, deletable);
|
| positions_[position++] = i;
|
| }
|
| }
|
|
|