Fixing a couple StackView bugs:

-> Making sure to update visuals every time adapter count changes
-> Fixing a clipping issue seen on some devices

Change-Id: I489395b5caaa06eb7187b2dac679b793bf54d7e1

2 files changed, 8 insertions, 2 deletions

diff --git a/core/java/android/widget/AdapterViewAnimator.java b/core/java/android/widget/AdapterViewAnimator.java
index 6eb06ad2e77a..e34a2041ba40 100644
--- a/core/java/android/widget/AdapterViewAnimator.java
+++ b/core/java/android/widget/AdapterViewAnimator.java
@@ -696,6 +696,8 @@ public abstract class AdapterViewAnimator extends AdapterView<Adapter>
                         mWhichChild = 0;
 
                         showOnly(mWhichChild, true);
+                    } else if (mOldItemCount != getCount()) {
+                        showOnly(mWhichChild, true);
                     }
                     refreshChildren();
                     requestLayout();
diff --git a/core/java/android/widget/StackView.java b/core/java/android/widget/StackView.java
index 9ec4b74c59e3..03c073c3d10b 100644
--- a/core/java/android/widget/StackView.java
+++ b/core/java/android/widget/StackView.java
@@ -454,9 +454,13 @@ public class StackView extends AdapterViewAnimator {
         canvas.getClipBounds(stackInvalidateRect);
         final int childCount = getChildCount();
         for (int i = 0; i < childCount; i++) {
-            LayoutParams lp = (LayoutParams) getChildAt(i).getLayoutParams();
+            final View child =  getChildAt(i);
+            LayoutParams lp = (LayoutParams) child.getLayoutParams();
+            if ((lp.horizontalOffset == 0 && lp.verticalOffset == 0) ||
+                    child.getAlpha() == 0f || child.getVisibility() != VISIBLE) {
+                lp.resetInvalidateRect();
+            }
             stackInvalidateRect.union(lp.getInvalidateRect());
-            lp.resetInvalidateRect();
         }
         canvas.save(Canvas.CLIP_SAVE_FLAG);
         canvas.clipRect(stackInvalidateRect, Region.Op.UNION);