GitLab

BubbleDialogDelegateView::Close() removes all its children. GridLayout
wasn't built for dynamically removing children, so its ColumnSets holds
on to stale pointers to Views previously in the hierarchy. This causes
problems when asynchronous calls are made to get the bubble's preferred
size before Widget::Close() asynchronously finishes.

This is solved by removing the GridLayout as layout manager before the
children are removed. Future updates to GridLayout would preferably
accommodate removal of children.

Bug: 1106422, 1130111
Change-Id: I2181856ae9669fbce9e6f3dfa347c468c7c5d567
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2425141
Auto-Submit: Peter Boström <pbos@chromium.org>
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809760}