commit a9265805dcec7d94cc78243dcd016765759f9a14
author Michael Jurka <mikejurka@google.com> Mon Jun 04 16:46:39 2012 -0700
committer Michael Jurka <mikejurka@google.com> Mon Jun 04 16:47:33 2012 -0700
tree b8848d28a6cee218befa3e9799387cf2216bc166
parent ca5a1489e22be07d2641e2a10456fa9e616fc4ed

Adding more logging for bug: 6499508

Change-Id: Iffc099e7de8b1d24bb33f1df6e17df7a099bfcb2

packages/SystemUI/src/com/android/systemui/recent/RecentsVerticalScrollView.java

diff --git a/packages/SystemUI/src/com/android/systemui/recent/RecentsVerticalScrollView.java b/packages/SystemUI/src/com/android/systemui/recent/RecentsVerticalScrollView.java
index 33d2a75..a1e36fb 100644
--- a/packages/SystemUI/src/com/android/systemui/recent/RecentsVerticalScrollView.java
+++ b/packages/SystemUI/src/com/android/systemui/recent/RecentsVerticalScrollView.java
@@ -75,6 +75,9 @@
     }
 
     private void addToRecycledViews(View v) {
+        if (mRecycledViews.contains(v)) {
+            throw new RuntimeException("Child was already recycled");
+        }
         if (mRecycledViews.size() < mNumItemsInOneScreenful) {
             mRecycledViews.add(v);
         }
@@ -104,12 +107,11 @@
             if (mRecycledViews.size() != 0) {
                 old = mRecycledViews.remove(mRecycledViews.size() - 1);
                 old.setVisibility(VISIBLE);
+                if (old.getParent() != null) {
+                    throw new RuntimeException("Recycled child has parent (i: " + i + ", recycled i: " + mRecycledViews.size());
+                }
             }
-
             final View view = mAdapter.getView(i, old, mLinearLayout);
-            if (view.getParent() != null) {
-                throw new RuntimeException("Recycled child has parent");
-            }
 
             if (mPerformanceHelper != null) {
                 mPerformanceHelper.addViewCallback(view);