commit bfeac0681b63091731c853fdafa3206642449a9d
author Winson Chung <winsonc@google.com> Wed Jun 06 15:56:08 2012 -0700
committer Winson Chung <winsonc@google.com> Wed Jun 06 15:57:27 2012 -0700
tree 849fcfccc0cdf250a82a8e387a47294d61174ae4
parent e6d3361ee85bf88f1f55970d7d80a20b31ea9ea7

Synchronizing on the new-apps list before modifying it. (Bug 6621553)

- Minor tweak to qsb padding
- Adding yet more logging for single-app-page issue

Change-Id: I4272dd7a70c9ac26f026b03a1f991f14844fdb9d

src/com/android/launcher2/Launcher.java

diff --git a/src/com/android/launcher2/Launcher.java b/src/com/android/launcher2/Launcher.java
index 9cae5e5..bbee787 100644
--- a/src/com/android/launcher2/Launcher.java
+++ b/src/com/android/launcher2/Launcher.java
@@ -3165,9 +3165,13 @@
                     View shortcut = createShortcut(info);
                     workspace.addInScreen(shortcut, item.container, item.screen, item.cellX,
                             item.cellY, 1, 1, false);
-                    if (newApps.contains(uri)) {
-                        newApps.remove(uri);
-
+                    boolean animateIconUp = false;
+                    synchronized (newApps) {
+                        if (newApps.contains(uri)) {
+                            animateIconUp = newApps.remove(uri);
+                        }
+                    }
+                    if (animateIconUp) {
                         // Prepare the view to be animated up
                         shortcut.setAlpha(0f);
                         shortcut.setScaleX(0f);