GitLab

The change in bounds_animator.cc looks scary (because it casts away const), but
is actually safe; view bounds are (correctly) stored in the data map as
pointer-to-non-const (since the animator will change the views' positions), but
this prevents lookup into the map by pointer-to-const.  The const cast allows
lookup, and the explicit use of find() instead of [] ensures the function is
actually const.

Bug: none
Change-Id: I3c217fc6ca931f6113cd661f1beeeb7e0a697475
Reviewed-on: https://chromium-review.googlesource.com/1121784
Commit-Queue: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#572009}