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

Unified Diff: third_party/WebKit/LayoutTests/accessibility/description-calc-inputs.html

Issue 1473833002: Fix bugs in accessible name calculation code exposed by Bryan Garaventa's test files. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@dumb_mistake
Patch Set: Fix android test expectation Created 5 years 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/description-calc-inputs.html
diff --git a/third_party/WebKit/LayoutTests/accessibility/description-calc-inputs.html b/third_party/WebKit/LayoutTests/accessibility/description-calc-inputs.html
index e82530d76a63f0429fbfa8c7362730f7fc3fe4f1..44c70e28f2cfe6935f2aad5108643afa8b3bdb0c 100644
--- a/third_party/WebKit/LayoutTests/accessibility/description-calc-inputs.html
+++ b/third_party/WebKit/LayoutTests/accessibility/description-calc-inputs.html
@@ -108,6 +108,25 @@ test(function(t) {
}, "placeholder overrides title as the accessible description.");
</script>
+<div class="container">
+ <input id="text8" type="text" aria-describedby="describedby8">
+ <p id="describedby8">
+ Described
+ <br>
+ By
+ </p>
+</div>
+
+<script>
+test(function(t) {
+ var axTextInput8 = accessibilityController.accessibleElementById("text8");
+ assert_equals(axTextInput8.name, "");
+ assert_equals(axTextInput8.nameFrom, "");
+ assert_equals(axTextInput8.description, "Described By");
+ assert_equals(axTextInput8.descriptionFrom, "relatedElement");
+}, "aria-describedby does not include newlines.");
+</script>
+
<script>
if (window.testRunner)
document.body.className = "hideAllContainers";

Powered by Google App Engine
This is Rietveld 408576698