Merge "Fixing concurrent modification exception when iterating through new apps list. (Bug 6621553)" into jb-dev

2012-06-25 14:45:32 -07:00
parent eb04476def 3df7dcf513
commit 9dc25f06a8
1 changed files with 4 additions and 3 deletions
@@ -3684,12 +3684,13 @@ public class Workspace extends SmoothPagedView
                    // Remove all queued items that match the same package
                    if (newApps != null) {
                        synchronized (newApps) {
-                            for (String intentStr : newApps) {
+                            Iterator<String> iter = newApps.iterator();
+                            while (iter.hasNext()) {
                                try {
-                                    Intent intent = Intent.parseUri(intentStr, 0);
+                                    Intent intent = Intent.parseUri(iter.next(), 0);
                                    String pn = ItemInfo.getPackageName(intent);
                                    if (packageNames.contains(pn)) {
-                                        newApps.remove(intentStr);
+                                        iter.remove();
                                    }
                                } catch (URISyntaxException e) {}
                            }