New issue
Advanced search Search tips

Issue 801292 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 660090



Sign in to add a comment

[Talk Back] Tapping on chapter books on page load reads " Page Content webview"

Project Member Reported by sbash...@chromium.org, Jan 11 2018

Issue description

MonoChrome Stable : 65.0.3317.0
Device : PIxel2/OPM1.171019.019

Pre req : Enable Talk Back ON
What steps will reproduce the problem?
(1) Launch Play Books
(2)Tap "A French Girl in New York"
(3)Tap Swipe to move to the Node selection .
(4) When entire page is selected, Observe the device

What is the expected result?
Page Content is spoken on selecting the entire page
What happens instead?
" Page Content webview" is read on selecting entire page

NOTE : This issue is reproduced in 63.0.3239.111 as well and hence it is not Regression

ADDITIONAL INFORMATION :L
1>Video Attached.
2.Screen cap Attached.
3.Bug Report attached
 
Bugreport,Video & Screen Capture attached =>  http://go/chrome-androidlogs1/7/801292
Labels: -talkback-te talk-back-te
Blockedon: 660090
Status: Started (was: Available)
This is WAI.

The title of the web page is "Page content":

<html>
  <head>
    <title>Page content</title>

It's supposed to be possible to make a WebView not ever get accessibility focus when you swipe. This will be fixed as part of  bug 660090 . Once that change lands, you should be able to remove the page title and then it will just skip over the entire page.

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 13 2018

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

commit a953ec895d88d23a76772b9ff5cce1a4a84348c8
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Wed Jun 13 16:46:35 2018

Android accessibility: skip the root element when it has no title

This change was originally landed in r432621 - the idea is that when a page
has no title, there's no reason we should allow the user to swipe to the
root element.

For some reason in r514252 this was inverted as part of making it possible
to jump to the last element on a page, but the new logic doesn't make
sense. Restore it to the previous behavior.

Bug: 661793,  660090 ,  801292 
Change-Id: Iba81d0429a04845f093c5f07449c26fa8514bda4
Reviewed-on: https://chromium-review.googlesource.com/1097953
Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566868}
[modify] https://crrev.com/a953ec895d88d23a76772b9ff5cce1a4a84348c8/content/browser/accessibility/browser_accessibility_android.cc
[modify] https://crrev.com/a953ec895d88d23a76772b9ff5cce1a4a84348c8/content/test/data/accessibility/html/isInteresting-expected-android.txt

Status: Fixed (was: Started)
verified using latest Chrome DEV v69.0.3475.0. Talkback issue is still persists.

Any webpage reads HTML page title as title + "webview".

Project Member

Comment 7 by bugdroid1@chromium.org, Jul 11

Labels: merge-merged-3440
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b099b45441859b14ed3b7ca81402af5c28fe2a92

commit b099b45441859b14ed3b7ca81402af5c28fe2a92
Author: Dominic Mazzoni <dmazzoni@chromium.org>
Date: Wed Jul 11 22:17:32 2018

Merge to M68: Android accessibility: skip the root element when it has no title

This change was originally landed in r432621 - the idea is that when a page
has no title, there's no reason we should allow the user to swipe to the
root element.

For some reason in r514252 this was inverted as part of making it possible
to jump to the last element on a page, but the new logic doesn't make
sense. Restore it to the previous behavior.

Bug: 661793,  660090 ,  801292 
Change-Id: Iba81d0429a04845f093c5f07449c26fa8514bda4
Reviewed-on: https://chromium-review.googlesource.com/1097953
Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#566868}(cherry picked from commit a953ec895d88d23a76772b9ff5cce1a4a84348c8)
Reviewed-on: https://chromium-review.googlesource.com/1134101
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/branch-heads/3440@{#649}
Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733}
[modify] https://crrev.com/b099b45441859b14ed3b7ca81402af5c28fe2a92/content/browser/accessibility/browser_accessibility_android.cc
[modify] https://crrev.com/b099b45441859b14ed3b7ca81402af5c28fe2a92/content/test/data/accessibility/html/isInteresting-expected-android.txt

Comment 8 Deleted

Comment 9 Deleted

Status: Verified (was: Fixed)
Verified on Pixel XL/OPM1.180608.001 having 68.0.3440.74 

Sign in to add a comment