New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 831918 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug



Sign in to add a comment

Scroll offsets are not maintained on a text field

Project Member Reported by nek...@chromium.org, Apr 12 2018

Issue description

This bug affects Zoomtext in a significant way, has been recently reported by the vendor and is within the Windows push, hence I am assigning a priority of 1.
Steps to reproduce:
data:text/html,<input size="50" maxlength="30" value="Very very very long piece of text">
1. Place the caret at the end of the input field.
2. Call IAccessible2.get_characterExtents(IA2_CARET_OFFSET, 0).
3. Observe that the X coordinate returned is outside the screen.

Cause:
1. scrollX and scrollY accessibility attributes are not present on the input field in the accessibility tree but are present on the div container that is at the root of the shadow DOM inside the input field. This means that they are ignored by the existing get_characterExtents logic in BrowserAccessibility.
2. The scroll offsets on the div at the root of the shadow DOM are not properly updated but scrollX stays set to 0 even after moving the caret left / right.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 1 2018

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

commit 4efd7e1319f90d6a919b51859301cd8316fbd42a
Author: Nektarios Paisios <nektar@chromium.org>
Date: Fri Jun 01 03:27:58 2018

Returns the correct character bounds when a text field overflows its contents

1. We were not using the unclipped bounds when calculating character bounds in BrowserAccessibility.
2. AXTree clipped bounds to 0 and dimension - 1, instead of container.x and container.dimension - 1.

R=dmazzoni@chromium.org

Bug:  831918 
Change-Id: I2634d7244146d491aa6ae7ff7abc48f0a661033b
Reviewed-on: https://chromium-review.googlesource.com/1044355
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#563519}
[modify] https://crrev.com/4efd7e1319f90d6a919b51859301cd8316fbd42a/content/browser/accessibility/accessibility_win_browsertest.cc
[modify] https://crrev.com/4efd7e1319f90d6a919b51859301cd8316fbd42a/content/browser/accessibility/browser_accessibility.cc
[modify] https://crrev.com/4efd7e1319f90d6a919b51859301cd8316fbd42a/ui/accessibility/ax_tree.cc

Status: Fixed (was: Started)

Sign in to add a comment