| commit | dd661f30d65756a59f48670749bd7d1426714aaf | |
| author | calamity <calamity@chromium.org> | |
| Tue, 4 Nov 2014 06:04:08 +0000 (3 22:04 -0800) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Tue, 4 Nov 2014 06:04:28 +0000 (4 06:04 +0000) | ||
| tree | d7adf2e4d64fefa5da9ecb96b09fa868f3cf368c | treesnapshot (tar.gz zip) |
| parent | 3064611cafd0464e4899dfdf7f485efe71f3ef57 | commitdiff |
Fix folder icon animation in app list.
This CL fixes the animation of folders opening in the app list. This
regressed in r301308 because the ContentsView layer moved, exposing
a latent issue with the calculation of the top icon animation target.
The fix here is to use the view bounds as the target rather than the
layer bounds. The layer bounds are in the coordinate system of the
parent layer which is not what we want.
BUG=429079
Review URL: https://codereview.chromium.org/694743002
Cr-Commit-Position: refs/heads/master@{#302579}
This CL fixes the animation of folders opening in the app list. This
regressed in r301308 because the ContentsView layer moved, exposing
a latent issue with the calculation of the top icon animation target.
The fix here is to use the view bounds as the target rather than the
layer bounds. The layer bounds are in the coordinate system of the
parent layer which is not what we want.
BUG=429079
Review URL: https://codereview.chromium.org/694743002
Cr-Commit-Position: refs/heads/master@{#302579}
| ui/app_list/views/top_icon_animation_view.cc | diffblobblamehistory |