4130a68a92b2c5264917d9ac37b51b74aa7993b9 - platform_frameworks_base

commit	4130a68a92b2c5264917d9ac37b51b74aa7993b9	[log] [tgz]
author	Jorim Jaggi <jjaggi@google.com>	Tue Jan 09 14:28:44 2018 +0100
committer	Jorim Jaggi <jjaggi@google.com>	Tue Jan 09 14:55:17 2018 +0100
tree	0b9422dbd2547057a49363eb16e4ccd18d671b68
parent	0d64cd33ba31d77f255c2240ea8c69b1a1b05144 [diff]

Fix SurfaceAnimator and SurfaceAnimationRunner tests

Since we marked mAnimator.mInitialized to true in the tests,
WM executed things from another thread during tests leading to
concurrency bugs.

Instead, we stub out addAfterPrepareSurfacesRunnable to a consumer
which executes the runnable directly during tests, avoiding the
need to let WM process animation frames.

Also attempts to fix flakyness in SurfaceAnimationRunner

Test: go/wm-smoke
Test: SurfaceAnimatorTest
Test: SurfaceAnimationRunnerTest

Change-Id: Ic9522e1afef6ce62667aefca80e58d6fb1db3424
Fixes: 71650763
Fixes: 71602314
Bug: 71719744

services/core/java/com/android/server/wm/AppWindowThumbnail.java[diff]
services/core/java/com/android/server/wm/SurfaceAnimator.java[diff]
services/core/java/com/android/server/wm/WindowAnimator.java[diff]
services/core/java/com/android/server/wm/WindowContainer.java[diff]
services/core/java/com/android/server/wm/WindowManagerService.java[diff]
services/tests/servicestests/src/com/android/server/wm/SurfaceAnimationRunnerTest.java[diff]
services/tests/servicestests/src/com/android/server/wm/SurfaceAnimatorTest.java[diff]
services/tests/servicestests/src/com/android/server/wm/WindowTestsBase.java[diff]

8 files changed

tree: 0b9422dbd2547057a49363eb16e4ccd18d671b68