GitLab

This CL fixes:
Consolidates TestShellObserver in window selector controller unittest and
TestOverviewObserver in window_selector_unittest. Removed two overlapping
tests.

Remove a TODO about bouncing a window, since a toast shows anyways.

Remove a enum for the header layout which is not needed anymore
because the header always fades in at the final location now. Previously
there was this enum to differentiate when the header animated with
the window, or when there was color on header which had to be slided
in.

Test: none
Bug: none
Change-Id: I7578b2809fda56b8f783875eef12f12ae6e57dd1
Reviewed-on: https://chromium-review.googlesource.com/c/1345536Reviewed-by: Xiaoqian Dai <xdai@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610917}