tree 37a8d613a93d07152e8b70a19913a0d6e2efcc05
parent 6e4710a96147bdf2b6bc7c52f3233e83e3624e9b
author Tom Sepez <tsepez@chromium.org> 1557336248 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1557336248 +0000

Use RemoveChild() in XFA_ReleaseLayoutItem.

Split off from https://pdfium-review.googlesource.com/c/pdfium/+/53571
to land separately.

Also fix the recursion in XFA_ReleaseLayoutItem() so that we operate
in an obvious depth-first manner (the old code had the potential to
fire duplicate OnLayoutItemRemoving() callbacks on the same node,
for example).

Change-Id: I3fb1e9bfde38f057c61fad14662143690459e0e5
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/54050
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>