Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't increment colorLayerCounter on surface migration

Browse Source 
If the TDA's surface is migrated mid transition then the colorLayerCounter is increment an additional time leading to a colorLayerCounter that is out of sync by +1 which means the color is never cleared.

Bug: 221775939

Test: Run repo steps from b/221775939 and make sure the background color doesn't presist
Change-Id: Ifc93d5482bad9591e942e4be91e0039daae743b1
This commit is contained in:
Pablo Gamito
2022-03-18 13:56:38 +00:00
parent cde55d305f
commit 5bf81c054c
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
services/core/java/com/android/server/wm/TaskDisplayArea.java
View File

@@ -824,9 +824,19 @@ final class TaskDisplayArea extends DisplayArea<WindowContainer> {
} }
void setBackgroundColor(@ColorInt int colorInt) { void setBackgroundColor(@ColorInt int colorInt) {
setBackgroundColor(colorInt, false /* restore */);
}
void setBackgroundColor(@ColorInt int colorInt, boolean restore) {
mBackgroundColor = colorInt; mBackgroundColor = colorInt;
Color color = Color.valueOf(colorInt); Color color = Color.valueOf(colorInt);
mColorLayerCounter++;
// We don't want to increment the mColorLayerCounter if we are restoring the background
// color after a surface migration because in that case the mColorLayerCounter already
// accounts for setting that background color.
if (!restore) {
mColorLayerCounter++;
}
// Only apply the background color if the TDA is actually attached and has a valid surface // Only apply the background color if the TDA is actually attached and has a valid surface
// to set the background color on. We still want to keep track of the background color state // to set the background color on. We still want to keep track of the background color state
@@ -855,7 +865,7 @@ final class TaskDisplayArea extends DisplayArea<WindowContainer> {
super.migrateToNewSurfaceControl(t); super.migrateToNewSurfaceControl(t);
if (mColorLayerCounter > 0) { if (mColorLayerCounter > 0) {
setBackgroundColor(mBackgroundColor); setBackgroundColor(mBackgroundColor, true /* restore */);
} }
if (mSplitScreenDividerAnchor == null) { if (mSplitScreenDividerAnchor == null) {