[wpt-sync] Sync PR 38899 - Revert "Consistency check between AXObjectCacheImpl and AXTreeSerializer tree"
Categories
(Testing :: web-platform-tests, task, P4)
Tracking
(firefox113 fixed)
| Tracking | Status | |
|---|---|---|
| firefox113 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
(Depends on 1 open bug, )
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 38899 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/38899
Details from upstream follow.
Alvin Jia <alvinjia@google.com> wrote:
Revert "Consistency check between AXObjectCacheImpl and AXTreeSerializer tree"
This reverts commit 505abab46074b91eb632e60d0d63b5e082282f83.
Reason for revert: breaking Crostini tests
Original change's description:
Consistency check between AXObjectCacheImpl and AXTreeSerializer tree
This is the lite version of checking the entire tree structure, only checking that the number of nodes is the same. It's easier to do and a more performant check. This CL only adds the checks. Four precursor CLs were landed to clean up object leaks and tree inconsistencies so that these checks would pass: AXMenuList (CL:4289642), Slots (CL:4294848), Image maps (CL:4291697), Inline text boxes (CL:4289593).
Follow up: a future check should be added to check for consistency between AXTreeSerializer and AXTree.
Bug: none
Change-Id: I0f33f9f7bdcd3f70cfce9182ab1111c7dfec6236
Cq-Do-Not-Cancel-Tryjobs: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4185240
Reviewed-by: David Tseng \<dtseng@chromium.org>
Commit-Queue: Aaron Leventhal \<aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1114055}Bug: none
Change-Id: Icddc46ec3868826a700ece8defa174c003cfbf7aReviewed-on: https://chromium-review.googlesource.com/4322517
WPT-Export-Revision: 9ecbd9ba72f62bc28ad48343dbe6514ce8b88a3e
| Assignee | ||
Comment 1•2 years ago
|
||
| Assignee | ||
Comment 2•2 years ago
|
||
| Assignee | ||
Comment 3•2 years ago
|
||
| Assignee | ||
Comment 5•2 years ago
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
Description
•