Issue metadata
Sign in to add a comment
|
Browser Android Accessibility scroll returns incorrect value |
||||||||||||||||||||||||
Issue descriptionUserAgent: 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:
,
Sep 8 2017
Please provide a sample testcase/url device deatils for further investigation.
,
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
,
Oct 12 2017
@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.
,
Oct 12 2017
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 |
|||||||||||||||||||||||||
Comment 1 by pbomm...@chromium.org
, Sep 7 2017