Revert "[Multipaste] Speak a11y announcement when delete button is focused"

This reverts commit a2c9272d.

Reason for revert: suspect this cl causes build failure https://ci.chromium.org/p/chromium/builders/ci/Linux%20ChromiumOS%20MSan%20Tests/21195

Original change's description:
> [Multipaste] Speak a11y announcement when delete button is focused
>
> This CL does two things:
> (1) Ignore the MenuItemView instance in accessibility events handling.
> Then we can customize accessibility behaviors of the multipaste menu
> in ClipboardHistoryItemView class.
>
> (2) When the delete button gets the pseudo focus, the accessibility
> side is notified.
>
> Bug: 1145773
> Change-Id: Ibd73b04637b89cc8200ca2f74721a10a44e857c4
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2513502
> Reviewed-by: Alex Newcomer <newcomer@chromium.org>
> Commit-Queue: Andrew Xu <andrewxu@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#824561}

TBR=newcomer@chromium.org,andrewxu@chromium.org

Change-Id: I999c30afd42fa4c703f506a536fd736bcbfb48df
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1146255
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2523237Reviewed-by: Tina Wang <tinazwang@chromium.org>
Commit-Queue: Tina Wang <tinazwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#824885}

ash/clipboard/clipboard_history_menu_model_adapter.cc

@@ -14,7 +14,6 @@
 #include "ui/base/data_transfer_policy/data_transfer_policy_controller.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/gfx/geometry/rect.h"
-#include "ui/views/accessibility/view_accessibility.h"
 #include "ui/views/controls/menu/menu_item_view.h"
 #include "ui/views/controls/menu/menu_runner.h"
 #include "ui/views/controls/menu/menu_types.h"
@@ -322,10 +321,6 @@ views::MenuItemView* ClipboardHistoryMenuModelAdapter::AppendMenuItem(
 
   views::MenuItemView* container = menu->AppendMenuItem(command_id);
 
-  // Ignore `container` in accessibility events handling. Let `item_view`
-  // handle.
-  container->GetViewAccessibility().OverrideIsIgnored(true);
-
   // Margins are managed by `ClipboardHistoryItemView`.
   container->SetMargins(/*top_margin=*/0, /*bottom_margin=*/0);
 

ash/clipboard/views/clipboard_history_item_view.cc

@@ -182,9 +182,6 @@ ClipboardHistoryItemView::ClipboardHistoryItemView(
     : clipboard_history_item_(clipboard_history_item), container_(container) {}
 
 void ClipboardHistoryItemView::Init() {
-  SetFocusBehavior(views::View::FocusBehavior::ACCESSIBLE_ONLY);
-  GetViewAccessibility().OverrideRole(ax::mojom::Role::kMenuItem);
-
   SetLayoutManager(std::make_unique<views::FillLayout>());
 
   // Ensures that MainButton is below any other child views.
@@ -325,21 +322,6 @@ void ClipboardHistoryItemView::SetPseudoFocus(PseudoFocus new_pseudo_focus) {
   pseudo_focus_ = new_pseudo_focus;
   contents_view_->delete_button()->SetVisible(ShouldShowDeleteButton());
   main_button_->SchedulePaint();
-  switch (pseudo_focus_) {
-    case PseudoFocus::kEmpty:
-      break;
-    case PseudoFocus::kMainButton:
-      NotifyAccessibilityEvent(ax::mojom::Event::kSelection,
-                               /*send_native_event=*/true);
-      break;
-    case PseudoFocus::kDeleteButton:
-      contents_view_->delete_button()->NotifyAccessibilityEvent(
-          ax::mojom::Event::kHover, /*send_native_event*/ true);
-      break;
-    case PseudoFocus::kMaxValue:
-      NOTREACHED();
-      break;
-  }
 }
 
 }  // namespace ash