diff options
| author | vollick@chromium.org <vollick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-03 21:38:08 +0000 |
|---|---|---|
| committer | vollick@chromium.org <vollick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-11-03 21:38:08 +0000 |
| commit | f5cd9e5d3c402226c8ff260eb63562df269f27ac (patch) | |
| tree | 81f71523b4497cd576a469e8dd5383a7cefeee27 /tools/checkbins | |
| parent | 6cd8a30576416c6e28debe4c811a22aa670e88db (diff) | |
| download | chromium_src-f5cd9e5d3c402226c8ff260eb63562df269f27ac.zip chromium_src-f5cd9e5d3c402226c8ff260eb63562df269f27ac.tar.gz chromium_src-f5cd9e5d3c402226c8ff260eb63562df269f27ac.tar.bz2 | |
Setting a layer property immediately should cancel any current animations.
This CL depends on http://codereview.chromium.org/8395046/
The fix boils down to two changes to the layer animator:
1. When calling Set*** we must start a zero duration animation and not short circuit. The animator needs to be able to update its state, and the preemption strategy needs to be applied.
2. Always be aware that progressing or aborting animation may affect the list of running animation and take appropriate precautions.
I have added a unit test to make sure the fix for 102413 doesn't regress and updated the existing unit tests for the new behavior (specifically, we now get notified that a sequence has ended whenever we call Set*** because of 1. above).
BUG=102413
TEST=compositor_unittests
Review URL: http://codereview.chromium.org/8387051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108546 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/checkbins')
0 files changed, 0 insertions, 0 deletions