GitLab

It seems that WindowState::UpdatePipState was overwriting window
visibility animation types set by other code. Modify it to be called
earlier and only to update the window visibility animation type on
transitions from and to PIP.

as well (properly uses FADE type animation)

Bug: 899432
Test: Animation to overview mode no longer has black backdrop animating
Test: Dismiss animation for PIP still works on closing the PIP window.
Change-Id: I9a575fbc55273cb4be7e26482f6f749e6d3ce15c
Reviewed-on: https://chromium-review.googlesource.com/c/1306971Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Eliot Courtney <edcourtney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604208}