GitLab

Explicit calls to View::SetVisible() outside of LayoutManager::Layout()
will cause the view's parent's layout manager (if there is one) to be
notified. This way, we can treat someone calling SetVisible() as a state
change for the layout manager, perpendicular to any changes the layout
manager itself makes to the visibility of child views (for example, if
it decides to hide them because there is not enough space to display
them).

Also, reordering child views invalidates layout. It should always have.

Bug: 898632
Change-Id: I47382e5efe348b63a7dda7b6e493f55ac5bd74fb
Reviewed-on: https://chromium-review.googlesource.com/c/1368274Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Dana Fried <dfried@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614832}