New issue
Advanced search Search tips

Issue 656779 link

Starred by 1 user
Status: Fixed
Owner:
dmazz...@chromium.org
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Android accessibility: whitespace-only nodes should be ignored

Project Member Reported by dmazz...@chromium.org, Oct 17 2016 
Try navigating through a page with <br>. TalkBack lands on the line break but says nothing.

<p>Before <a href="#">Link</a> <br> <a href="#">Another link</a> After</p>
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 24 2016 

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

commit 100ba42f5f6b4ec409c5106749a65bdbb04c4328
Author: dmazzoni <dmazzoni@chromium.org>
Date: Mon Oct 24 22:25:17 2016

Android accessibility should not treat whitespace-only nodes as interesting.

When linearly navigating through nodes in the accessibility tree, don't treat
a leaf node with text as interesting if the text is only whitespace.
Also fix an issue where requesting the first "interesting" node
from the root of the tree could sometimes return the root itself, leading to
a cycle.

BUG= 656779 

Review-Url: https://codereview.chromium.org/2427813002
Cr-Commit-Position: refs/heads/master@{#427172}

[modify] https://crrev.com/100ba42f5f6b4ec409c5106749a65bdbb04c4328/content/browser/accessibility/browser_accessibility_android.cc
[modify] https://crrev.com/100ba42f5f6b4ec409c5106749a65bdbb04c4328/content/browser/accessibility/one_shot_accessibility_tree_search.cc
[modify] https://crrev.com/100ba42f5f6b4ec409c5106749a65bdbb04c4328/content/browser/accessibility/one_shot_accessibility_tree_search_unittest.cc

Comment 2 by dmazz...@chromium.org, Oct 24 2016

Status: Fixed (was: Started)
►

Sign in to add a comment