Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Issue 1825733002: Revert of Exposed the children of elements with role textbox in order to make rich text information available… (Closed)

Created:
4 years, 9 months ago by newt (away)
Modified:
4 years, 9 months ago
Reviewers:
nektarios, dmazzoni
CC:
nektarios, aboxhall+watch_chromium.org, aboxhall, blink-reviews, chromium-reviews, darin-cc_chromium.org, dmazzoni+watch_chromium.org, dmazzoni, dtseng+watch_chromium.org, jam, je_julie, nektar+watch_chromium.org, yuzo+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Exposed the children of elements with role textbox in order to make rich text information available… (patchset #4 id:60001 of https://codereview.chromium.org/1821723002/ ) Reason for revert: DumpAccessibilityTreeTest.AccessibilityAriaTextboxWithRichText is now failing on most Android bots. See crbug.com/597157 Original issue's description: > Exposed the children of elements with role textbox in order to make rich text information available to Windows screen readers. > > If we have an HTML element with role="textbox" we correctly assign it an IA role of ROLE_SYSTEM_TEXT. This is the same as what FireFox does. > However, contrary to an <input> or <textarea> element, we should expose its children so that any rich text information would be available to Windows screen readers. > Note that we also keep exposing a flatten representation of the children's text in the value attribute because Jaws and older versions of VoiceOver require it. > BUG=542998 > TESTED:browser tests, Blink layout tests, manually with Jaws and NVDA with <input>, content editable and content editable with role="textbox" > R=dmazzoni@chromium.org > > Committed: https://crrev.com/9a702688af0b856d52e96f1ce9b5346714ce8985 > Cr-Commit-Position: refs/heads/master@{#382526} TBR=dmazzoni@chromium.org,nektar@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=542998 Committed: https://crrev.com/0fc016d5f294b89638acdd640bf677d2349cd7ec Cr-Commit-Position: refs/heads/master@{#382774}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -78 lines) Patch
M content/browser/accessibility/browser_accessibility.cc View 3 chunks +5 lines, -7 lines 0 comments Download
M content/browser/accessibility/browser_accessibility_win.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/accessibility/browser_accessibility_win_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/accessibility/dump_accessibility_tree_browsertest.cc View 1 chunk +1 line, -6 lines 0 comments Download
D content/test/data/accessibility/aria/aria-textbox-with-rich-text.html View 1 chunk +0 lines, -28 lines 0 comments Download
D content/test/data/accessibility/aria/aria-textbox-with-rich-text-expected-android.txt View 1 chunk +0 lines, -5 lines 0 comments Download
D content/test/data/accessibility/aria/aria-textbox-with-rich-text-expected-mac.txt View 1 chunk +0 lines, -11 lines 0 comments Download
D content/test/data/accessibility/aria/aria-textbox-with-rich-text-expected-win.txt View 1 chunk +0 lines, -11 lines 0 comments Download
M third_party/WebKit/LayoutTests/accessibility/is-richly-editable.html View 2 chunks +4 lines, -4 lines 0 comments Download
M third_party/WebKit/LayoutTests/accessibility/is-richly-editable-expected.txt View 2 chunks +2 lines, -3 lines 0 comments Download
M third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp View 1 chunk +3 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (1 generated)
newt (away)
4 years, 9 months ago (2016-03-23 02:31:32 UTC) #1
Created Revert of Exposed the children of elements with role textbox in order to
make rich text information available…

Powered by Google App Engine
This is Rietveld 408576698