Remove items on long press in keyboard accessory.

chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.cc

 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/android/autofill/autofill_keyboard_accessory_view.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "chrome/browser/android/resource_mapper.h"
 #include "chrome/browser/ui/android/window_android_helper.h"
 #include "chrome/browser/ui/autofill/autofill_popup_controller.h"
 #include "components/autofill/core/browser/suggestion.h"
 #include "grit/components_strings.h"
 #include "jni/AutofillKeyboardAccessoryBridge_jni.h"
 #include "ui/android/view_android.h"
 #include "ui/android/window_android.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/geometry/rect.h"
 
 namespace autofill {
 
 AutofillKeyboardAccessoryView::AutofillKeyboardAccessoryView(
     AutofillPopupController* controller)
-    : controller_(controller) {
+    : controller_(controller),
+      deleting_index_(-1) {
   JNIEnv* env = base::android::AttachCurrentThread();
   java_object_.Reset(Java_AutofillKeyboardAccessoryBridge_create(env));
 }
 
 AutofillKeyboardAccessoryView::~AutofillKeyboardAccessoryView() {
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_AutofillKeyboardAccessoryBridge_resetNativeViewPointer(
       env, java_object_.obj());
 }
 
@@ skipping 23 matching lines @@
                                                                          count);
 
   for (size_t i = 0; i < count; ++i) {
     const autofill::Suggestion& suggestion = controller_->GetSuggestionAt(i);
     int android_icon_id = 0;
     if (!suggestion.icon.empty()) {
       android_icon_id = ResourceMapper::MapFromChromiumId(
           controller_->GetIconResourceID(suggestion.icon));
     }
 
+    bool deletable =
+        controller_->GetRemovalConfirmationText(i, nullptr, nullptr);
     Java_AutofillKeyboardAccessoryBridge_addToAutofillSuggestionArray(
         env, data_array.obj(), i,
         base::android::ConvertUTF16ToJavaString(env, suggestion.value).obj(),
         base::android::ConvertUTF16ToJavaString(env, suggestion.label).obj(),
-        android_icon_id, suggestion.frontend_id);
+        android_icon_id, suggestion.frontend_id, deletable);
   }
 
   Java_AutofillKeyboardAccessoryBridge_show(
       env, java_object_.obj(), data_array.obj(), controller_->IsRTL());
 }
 
 void AutofillKeyboardAccessoryView::SuggestionSelected(JNIEnv* env,
                                                        jobject obj,
                                                        jint list_index) {
   // Race: Hide() may have already run.
   if (controller_)
     controller_->AcceptSuggestion(list_index);
 }
 
+void AutofillKeyboardAccessoryView::DeletionRequested(JNIEnv* env,
+                                                      jobject obj,
+                                                      jint list_index) {
+  if (!controller_)
+    return;
+
+  base::string16 confirmation_title, confirmation_body;
+  if (!controller_->GetRemovalConfirmationText(list_index, &confirmation_title,
+                                               &confirmation_body)) {
+    return;
+  }
+
+  deleting_index_ = list_index;

[Yaron, 2015/08/14 18:13:42]

i would much prefer it the way newt@ suggested of having it stored in the tag.
Also, is there not a better key that can be used (or is the index stable?). I
worry about the racey case of the backend getting updated in between the prompt
and ack.

[please use gerrit instead, 2015/08/14 20:23:31]

Done.
Webpage interaction is blocked after DeletionRequested(). Autofill updates only
on user gestures. Therefore, the suggestions array will remain unchanged between
DeletionRequested() and DeletionConfirmed(). Thus the index is stable.

+  Java_AutofillKeyboardAccessoryBridge_confirmDeletion(
+      env, java_object_.obj(),
+      base::android::ConvertUTF16ToJavaString(env, confirmation_title).obj(),
+      base::android::ConvertUTF16ToJavaString(env, confirmation_body).obj());
+}
+
+void AutofillKeyboardAccessoryView::DeletionConfirmed(JNIEnv* env,
+                                                      jobject obj) {
+  if (!controller_)
+    return;
+
+  CHECK_GE(deleting_index_, 0);
+  controller_->RemoveSuggestion(deleting_index_);
+}
+
 void AutofillKeyboardAccessoryView::ViewDismissed(JNIEnv* env, jobject obj) {
   if (controller_)
     controller_->ViewDestroyed();
 
   delete this;
 }
 
 void AutofillKeyboardAccessoryView::InvalidateRow(size_t) {
 }
 
 // static
 bool AutofillKeyboardAccessoryView::RegisterAutofillKeyboardAccessoryView(
     JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace autofill