diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-24 21:25:01 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-24 21:25:01 +0000 |
| commit | 701d7d6c743985431d1aab03da77669d09d1886d (patch) | |
| tree | 86a1ef38fc710b9899fade0c239ab521907e8424 /ash/wm/window_animations.h | |
| parent | 78322e8767f52007ba6e3ce3a222cb627998d90e (diff) | |
| download | chromium_src-701d7d6c743985431d1aab03da77669d09d1886d.zip chromium_src-701d7d6c743985431d1aab03da77669d09d1886d.tar.gz chromium_src-701d7d6c743985431d1aab03da77669d09d1886d.tar.bz2 | |
Fixes crash introduced @ 153047 (you can hit crash by maximizing a
window). The cross fade code deletes the layer when the animation
finishes. The newly added code was accessing members after the
animation finished and the animator was deleted, resulting in the
crash. Since I'm sure this will come up more in the future I've
restructured the code to allow for deletion when calling out like
this.
The cross fade test exercises this code path now, but I'll see about a
more focused tests shortly.
BUG=129033
TEST=covered by tests.
R=vollick@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10874064
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153291 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash/wm/window_animations.h')
| -rw-r--r-- | ash/wm/window_animations.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/ash/wm/window_animations.h b/ash/wm/window_animations.h index 9ddbbac..192b0e7 100644 --- a/ash/wm/window_animations.h +++ b/ash/wm/window_animations.h @@ -119,10 +119,6 @@ ASH_EXPORT base::TimeDelta GetCrossFadeDuration(const gfx::Rect& old_bounds, ASH_EXPORT bool AnimateOnChildWindowVisibilityChanged( aura::Window* window, bool visible); -// Delay the old layer deletion so that test can verify the behavior of -// old layer. -ASH_EXPORT void SetDelayedOldLayerDeletionInCrossFadeForTest(bool value); - } // namespace internal } // namespace ash |