commit 9bfba9c157e4a5b1325a60d2122be2bf034a5a2a
author Selim Cinek <cinek@google.com> Mon Aug 08 15:20:06 2016 -0700
committer Selim Cinek <cinek@google.com> Mon Aug 08 15:20:06 2016 -0700
tree db371186f0080f37fd194acaacc68b6aae21784f
parent d393d5c1e51e61b7ac61d3415586b696ee5c5f2a

Made background updater more robust to state changes

With raceconditions, the background of notifications could
become colored when dozing.

Change-Id: Ib39f873e6dce40e97a8df06dec281733a8ae2fd9
Fixes: 29331596

packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java

diff --git a/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java b/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java
index f72e50b..e60a809 100644
--- a/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java
+++ b/packages/SystemUI/src/com/android/systemui/statusbar/stack/NotificationStackScrollLayout.java
@@ -1963,9 +1963,9 @@
     }
 
     private void applyCurrentBackgroundBounds() {
-        if (!mFadingOut) {
-            mScrimController.setExcludedBackgroundArea(mCurrentBounds);
-        }
+        mScrimController.setExcludedBackgroundArea(
+                mFadingOut || mParentFadingOut || mAmbientState.isDark() ? null
+                        : mCurrentBounds);
         invalidate();
     }
 
@@ -3843,11 +3843,7 @@
     }
 
     private void updateFadingState() {
-        if (mFadingOut || mParentFadingOut || mAmbientState.isDark()) {
-            mScrimController.setExcludedBackgroundArea(null);
-        } else {
-            applyCurrentBackgroundBounds();
-        }
+        applyCurrentBackgroundBounds();
         updateSrcDrawing();
     }