New issue
Advanced search Search tips

Issue 887534 link

Starred by 1 user
Status: Fixed
Owner:
aleventhal@chromium.org
Closed: Sep 25
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 3
Type: Bug
Team-Accessibility



Sign in to add a comment

Caret hide events missing when focus moves to non-editable element

Project Member Reported by aleventhal@chromium.org, Sep 20 
Steps:
1. Watch IA2 events
2. Focus an input and then tab to a button (e.g. on data:text/html,<form><input type="text" name="zzz"><input type="submit"></form>)

Caret hide event should occur, but does not.
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 25 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7801fa45b16cfd270f75024e8e567e2427c568de

commit 7801fa45b16cfd270f75024e8e567e2427c568de
Author: Aaron Leventhal <aleventhal@chromium.org>
Date: Tue Sep 25 19:56:27 2018

Missing caret hide events in Windows accessibility support

Fix: Plumb through IME state change use to notify of caret change

Bug:  887534 
Change-Id: If472661b199afcbc8b30b7cb99e6d83b15114942
Reviewed-on: https://chromium-review.googlesource.com/1236735
Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#594065}
[modify] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/content/browser/accessibility/dump_accessibility_events_browsertest.cc
[add] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/content/test/data/accessibility/event/caret-hide-expected-win.txt
[add] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/content/test/data/accessibility/event/caret-hide.html
[modify] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/ui/base/ime/input_method_win_imm32.cc
[modify] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/ui/base/ime/input_method_win_tsf.cc
[modify] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/ui/views/accessibility/ax_system_caret_win_interactive_uitest.cc
[modify] https://crrev.com/7801fa45b16cfd270f75024e8e567e2427c568de/ui/views/win/hwnd_message_handler.cc

Comment 2 by aleventhal@chromium.org, Sep 25

Status: Fixed (was: Assigned)
►

Sign in to add a comment