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

Unified Diff: third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html

Issue 1435113003: Make use of new AX name calc in Chromium. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix issue with ariaTextAlternative Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html
diff --git a/third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html b/third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html
index 38e04bdc0b89ec07d4f4039eec9227f0531eb2fa..39a7c7ef97a7c7ef7751f820bd1284abbd499063 100644
--- a/third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html
+++ b/third_party/WebKit/LayoutTests/accessibility/inline-text-bounds-for-range.html
@@ -25,9 +25,7 @@ A resumé, it says, is about the most massively useful thing an interstellar hit
var axParagraph = accessibilityController.accessibleElementById('paragraph');
var axStaticText = axParagraph.childAtIndex(0);
- // The first characters of stringValue are 'AXValue: ' - use substr to get rid of those.
- // The accessible text shouldn't have any spaces.
- var text = axStaticText.stringValue.substr(9);
+ var text = axStaticText.name;
shouldBe("text.length", "185");
debug("Accessible text: \"" + text + "\"");
@@ -35,7 +33,7 @@ A resumé, it says, is about the most massively useful thing an interstellar hit
var appendedInlineText = '';
for (var i = 0; i < axStaticText.childrenCount; i++) {
var axInlineTextBox = axStaticText.childAtIndex(i);
- appendedInlineText += axInlineTextBox.stringValue.substr(9);
+ appendedInlineText += axInlineTextBox.name;
}
shouldBe("appendedInlineText", "text");

Powered by Google App Engine
This is Rietveld 408576698