Skip to content

GitLab

T
tangled
Commit 686b4210 authored by Nektarios Paisios's avatar Nektarios Paisios Committed by Commit Bot
Browse files
Options

Revert "Added back accessibility word boundaries Blink web test" 

This reverts commit d40c7abd.

Reason for revert: Broke Linux and Win builders

Original change's description:
> Added back accessibility word boundaries Blink web test
> 
> In https://crrev.com/c/2237930/ the Blink web test for accessibility word boundaries
> was removed due to an issue with some, but not all, of the try bots.
> 
> TBR=dmazzoni@chromium.org, aleventhal@chromium.org
> 
> AX-Relnotes: n/a.
> Bug: 1028830
> Change-Id: Ib5b05c68203a627f52081a6d423ed50a890f3566
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2275885
> Reviewed-by: Nektarios Paisios <nektar@chromium.org>
> Auto-Submit: Nektarios Paisios <nektar@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#784438}

TBR=dmazzoni@chromium.org,nektar@chromium.org,aleventhal@chromium.org

Change-Id: Ic25df1619a2c665290ca54d405a2cc8b5c1f7eb2
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1028830
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2277738Reviewed-by: default avatarNektarios Paisios <nektar@chromium.org>
Commit-Queue: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#784468}
parent 67960cdc
Hide whitespace changes
Inline Side-by-side
Showing with 0 additions and 94 deletions
third_party/blink/web_tests/accessibility/inline-text-word-boundaries-expected.txt deleted 100644 → 0
View file @ 67960cdc
Tests that we can compute word boundaries for any character index from the accessibility tree.
On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
Accessible text: "Ace ten twenty-one thirty-five. xxxxxxxxxxxxxxxxxxx"
Character 0: 'A' word=(0, 3): 'Ace'
Character 1: 'c' word=(0, 3): 'Ace'
Character 2: 'e' word=(0, 3): 'Ace'
Character 3: ' ' word=(4, 7): 'ten'
Character 4: 't' word=(4, 7): 'ten'
Character 5: 'e' word=(4, 7): 'ten'
Character 6: 'n' word=(4, 7): 'ten'
Character 7: ' ' word=(8, 14): 'twenty'
Character 8: 't' word=(8, 14): 'twenty'
Character 9: 'w' word=(8, 14): 'twenty'
Character 10: 'e' word=(8, 14): 'twenty'
Character 11: 'n' word=(8, 14): 'twenty'
Character 12: 't' word=(8, 14): 'twenty'
Character 13: 'y' word=(8, 14): 'twenty'
Character 14: '-' word=(14, 15): '-'
Character 15: 'o' word=(15, 18): 'one'
Character 16: 'n' word=(15, 18): 'one'
Character 17: 'e' word=(15, 18): 'one'
Character 18: ' ' word=(15, 18): 'one'
Character 19: 't' word=(19, 25): 'thirty'
Character 20: 'h' word=(19, 25): 'thirty'
Character 21: 'i' word=(19, 25): 'thirty'
Character 22: 'r' word=(19, 25): 'thirty'
Character 23: 't' word=(19, 25): 'thirty'
Character 24: 'y' word=(19, 25): 'thirty'
Character 25: '-' word=(25, 26): '-'
Character 26: 'f' word=(26, 30): 'five'
Character 27: 'i' word=(26, 30): 'five'
Character 28: 'v' word=(26, 30): 'five'
Character 29: 'e' word=(26, 30): 'five'
Character 30: '.' word=(30, 31): '.'
Character 31: ' ' word=(30, 31): '.'
Character 32: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 33: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 34: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 35: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 36: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 37: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 38: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 39: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 40: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 41: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 42: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 43: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 44: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 45: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 46: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 47: 'x' word=(32, 48): 'xxxxxxxxxxxxxxxx'
Character 48: 'x' word=(48, 51): 'xxx'
Character 49: 'x' word=(48, 51): 'xxx'
Character 50: 'x' word=(48, 51): 'xxx'
PASS successfullyParsed is true
TEST COMPLETE
Ace ten twenty-one thirty-five. xxxxxxxxxxxxxxxxxxx
third_party/blink/web_tests/accessibility/inline-text-word-boundaries.html deleted 100644 → 0
View file @ 67960cdc
<!DOCTYPE html>
<script src="../resources/js-test.js"></script>
<!-- Note: the xxxxxx... should wrap onto another line on all platforms. -->
<p id="paragraph" style="width: 8em; word-wrap: break-word;">
Ace ten twenty-one thirty-five. xxxxxxxxxxxxxxxxxxx
</p>
<script>
'use strict';
description('Tests that we can compute word boundaries for any character index from the accessibility tree.');
// Note: All that matters is that characters within actual words are mapped correctly.
// Whether a character between words is mapped to the previous or next word is arbitrary;
// all that matters is that we expose the start and end of real words.
if (window.accessibilityController) {
const axParagraph = accessibilityController.accessibleElementById('paragraph');
const axStaticText = axParagraph.childAtIndex(0);
const text = axStaticText.name;
debug('Accessible text: "' + text + '"');
for (let i = 0; i < text.length; i++) {
// Get the bounds from the accessibility tree.
let wordStart = axStaticText.wordStart(i);
let wordEnd = axStaticText.wordEnd(i);
debug("Character " + i + ": '" + text.substr(i, 1) +
"' word=(" + wordStart + ", " + wordEnd + "): '" +
text.substr(wordStart, wordEnd - wordStart) + "'");
}
}
</script>
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment