Don't fire a FOCUS accessibility event on a View that doesn't have focus.

ash/wm/overview/window_selector_item.cc

 // Copyright 2013 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 "ash/wm/overview/window_selector_item.h"
 
 #include <algorithm>
 #include <vector>
 
 #include "ash/screen_util.h"
@@ skipping 195 matching lines @@
   if (in_bounds_update_ || target_bounds_.IsEmpty())
     return;
   base::AutoReset<bool> auto_reset_in_bounds_update(&in_bounds_update_, true);
   gfx::Rect inset_bounds(target_bounds_);
   inset_bounds.Inset(kWindowMargin, kWindowMargin);
   SetItemBounds(inset_bounds, OverviewAnimationType::OVERVIEW_ANIMATION_NONE);
   UpdateCloseButtonLayout(OverviewAnimationType::OVERVIEW_ANIMATION_NONE);
 }
 
 void WindowSelectorItem::SendFocusAlert() const {
-  window_label_button_view_->NotifyAccessibilityEvent(ui::AX_EVENT_FOCUS, true);
+  window_label_button_view_->NotifyAccessibilityEvent(
+      ui::AX_EVENT_SELECTION, true);

[sky, 2016/02/14 16:21:52]

If the name of this method is correct, why send selection rather than focus?

[dmazzoni, 2016/02/17 00:25:18]

The name isn't right, the view isn't actually focused. Renamed the method to
SendAccessibleSelectionEvent.

 }
 
 void WindowSelectorItem::SetDimmed(bool dimmed) {
   dimmed_ = dimmed;
   SetOpacity(dimmed ? kDimmedItemOpacity : 1.0f);
 }
 
 void WindowSelectorItem::ButtonPressed(views::Button* sender,
                                        const ui::Event& event) {
   if (sender == close_button_) {
@@ skipping 120 matching lines @@
       close_button_transform);
 }
 
 void WindowSelectorItem::UpdateCloseButtonAccessibilityName() {
   close_button_->SetAccessibleName(l10n_util::GetStringFUTF16(
       IDS_ASH_OVERVIEW_CLOSE_ITEM_BUTTON_ACCESSIBLE_NAME,
       GetWindow()->title()));
 }
 
 }  // namespace ash