Skip to content

Commit

Permalink
[A11y] Address possibility of nodeless object with content-visibility…
Browse files Browse the repository at this point in the history
… rule

Note that the new test does not exercise all of the added code, because
an AXObject never existed for the nodeless layout object in the
scrollbar. However, rather than hitting a CHECK(), the code will now
continue and ensure that if there is a subtree of AXObjects based
only on layout objects, that they are removed in this case.

Fixed: 1481892
Change-Id: Iebacc810852c0a34e6f743a821957360bcfd69d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4860045
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1196125}
  • Loading branch information
aleventhal authored and Lightning00Blade committed Dec 11, 2023
1 parent 5c1775a commit 757ce69
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions accessibility/crashtests/null-node.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
<!DOCTYPE html>
<style>
::-webkit-scrollbar-track, ::-webkit-scrollbar {
color: red;
}
::-webkit-scrollbar-track:horizontal:disabled {
content-visibility: hidden;
}
</style>
<main style="height: 100px; width: 100px; overflow: scroll;"></main>

0 comments on commit 757ce69

Please sign in to comment.