GitLab

This reverts commit e5ab7cc7.

Reason for revert: The DeleteClientSubtree() changes can lead to crash.

Original change's description:
> Make AXTreeSerializer safer
>
> The tree serializer should not crash even if the given tree has invalid
> nodes. These changes don't alter behavior but removed possibilities for
> accessing dangling pointers.
>
> Bug: 981311
> Change-Id: I8f2e55771c12a03955f7632335bea347b6a5af30
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1707949
> Commit-Queue: Nektarios Paisios <nektar@chromium.org>
> Reviewed-by: Nektarios Paisios <nektar@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#678670}

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

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 981311
Change-Id: I1d00c05fb0af2ec48d645e475cfcceb9421bd5d1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1713813
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#679806}