commit 9e6a0a20d40675ef98c7fdb8892cf34b90152f7a
author Winson Chung <winsonc@google.com> Tue Aug 27 11:58:12 2013 -0700
committer Winson Chung <winsonc@google.com> Tue Aug 27 15:54:46 2013 -0700
tree 5c3a9fe04f8de3e466043c6a2fe56379c5e63fc2
parent 53fc06d5d6d4b57e824490300bf62dbed54d9ac7

Clearing workspace screen data structures along with pages.

Change-Id: Ib6c4522cd0c0c06dd6fd327be3621a6de0778f9c

src/com/android/launcher3/LauncherModel.java

diff --git a/src/com/android/launcher3/LauncherModel.java b/src/com/android/launcher3/LauncherModel.java
index 8676b09..c1754b4 100644
--- a/src/com/android/launcher3/LauncherModel.java
+++ b/src/com/android/launcher3/LauncherModel.java
@@ -283,7 +283,7 @@
                                     final Callbacks callbacks) {
         Log.w(TAG, "10249126 - addAndBindAddedApps()");
         if (added.isEmpty()) {
-            throw new RuntimeException("EMPTY ADDED ARRAY?");
+            return;
         }
         // Process the newly added applications and add them to the database first
         Runnable r = new Runnable() {
@@ -979,6 +979,7 @@
      * a list of screen ids in the order that they should appear.
      */
     void updateWorkspaceScreenOrder(Context context, final ArrayList<Long> screens) {
+        Log.w(TAG, "10249126 - updateWorkspaceScreenOrder()");
         final ArrayList<Long> screensCopy = new ArrayList<Long>(screens);
         final ContentResolver cr = context.getContentResolver();
         final Uri uri = LauncherSettings.WorkspaceScreens.CONTENT_URI;
@@ -988,10 +989,17 @@
         while (iter.hasNext()) {
             long id = iter.next();
             if (id < 0) {
+                Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - remove: " + id + ")");
                 iter.remove();
             }
         }
 
+        // Dump the screens copy
+        Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - screensCopy");
+        for (Long l : screensCopy) {
+            Log.w(TAG, "10249126\t- " + l);
+        }
+
         Runnable r = new Runnable() {
             @Override
             public void run() {
@@ -1004,10 +1012,18 @@
                     long screenId = screensCopy.get(i);
                     v.put(LauncherSettings.WorkspaceScreens._ID, screenId);
                     v.put(LauncherSettings.WorkspaceScreens.SCREEN_RANK, i);
-                    Log.w(TAG, "10249126 - updateWorkspaceScreenOrder(" + screenId + ", " + i + ")");
+                    Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - add: " +
+                            screenId + ", " + i + ")");
                     values[i] = v;
                 }
                 cr.bulkInsert(uri, values);
+
+                // Dump the sBgWorkspaceScreens
+                Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - sBgWorkspaceScreens");
+                for (Long l : sBgWorkspaceScreens) {
+                    Log.w(TAG, "10249126\t- " + l);
+                }
+
                 sBgWorkspaceScreens.clear();
                 sBgWorkspaceScreens.addAll(screensCopy);
             }
@@ -1915,6 +1931,13 @@
                         }
                     }
                     Collections.sort(sBgWorkspaceScreens);
+
+                    // Dump the sBgWorkspaceScreens
+                    Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - sBgWorkspaceScreens");
+                    for (Long l : sBgWorkspaceScreens) {
+                        Log.w(TAG, "10249126\t- " + l);
+                    }
+
                     mApp.getLauncherProvider().updateMaxScreenId(maxScreenId);
                     updateWorkspaceScreenOrder(context, sBgWorkspaceScreens);
 
@@ -1948,6 +1971,13 @@
                     // If there are any empty screens remove them, and update.
                     if (unusedScreens.size() != 0) {
                         sBgWorkspaceScreens.removeAll(unusedScreens);
+
+                        // Dump the sBgWorkspaceScreens
+                        Log.w(TAG, "10249126 - updateWorkspaceScreenOrder - sBgWorkspaceScreens");
+                        for (Long l : sBgWorkspaceScreens) {
+                            Log.w(TAG, "10249126\t- " + l);
+                        }
+
                         updateWorkspaceScreenOrder(context, sBgWorkspaceScreens);
                     }
                 }
@@ -2101,6 +2131,13 @@
         private void bindWorkspaceScreens(final Callbacks oldCallbacks,
                 final ArrayList<Long> orderedScreens) {
             Log.w(TAG, "10249126 - bindWorkspaceScreens()");
+
+            // Dump the orderedScreens
+            Log.w(TAG, "10249126 - orderedScreens");
+            for (Long l : sBgWorkspaceScreens) {
+                Log.w(TAG, "10249126\t- " + l);
+            }
+
             final Runnable r = new Runnable() {
                 @Override
                 public void run() {