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

Issue 763014 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Browser Android Accessibility scroll returns incorrect value

Project Member Reported by ibobra@google.com, Sep 7 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/61.0.3163.79 Safari/537.36

Steps to reproduce the problem:
BrowserAccessibilityAndroid::Scroll(int direction) should not return true if the position doesn't change.

What is the expected behavior?
It should return false.

What went wrong?
N/A

Did this work before? N/A 

Chrome version: 61.0.3163.79  Channel: stable
OS Version: 
Flash Version:
 
Labels: -OS-Linux OS-Android
based on offline chat with Isha this is specific to Android hence tagging the bug as Android.
Labels: Needs-triage-Mobile
Please provide a sample testcase/url device deatils for further investigation.
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 12 2017

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

commit 6dd5a34b1d4fd761152d93e251546cd84730187b
Author: Isha Bobra <ibobra@google.com>
Date: Tue Sep 12 18:31:01 2017

Android Accessibility: fix return value for scroll actions

There is a chance that a race condition occurs where the page returns canScrollForward/backward/up/down/left/right as true but actually the page cannot scroll further.
In those cases the scroll function in browser android accessibility should return false instead of
true.

Bug:  763014 
Change-Id: I777b5fb562713e0bd835a354adab9cbf86fcd853
Reviewed-on: https://chromium-review.googlesource.com/661718
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#501340}
[modify] https://crrev.com/6dd5a34b1d4fd761152d93e251546cd84730187b/content/browser/accessibility/browser_accessibility_android.cc

Cc: msrchandra@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org sandeepkumars@chromium.org
Components: UI>Accessibility
Labels: Needs-Feedback
@ibobra -- Could you please provide any sample testcase/url as per Comment# 2 which would help us in triaging the issue further.
Thanks in Advance.

Comment 5 by ibobra@google.com, Oct 12 2017

Status: Fixed (was: Unconfirmed)
I have already landed a fix for this issue. This bug can be closed and no triage is required :)

Sign in to add a comment