Add a log message and some todos.

Change-Id: I948997d169d1833f2b61c620a02dbccae535e8de

1 files changed, 7 insertions, 0 deletions

diff --git a/src/com/android/launcher2/LauncherModel.java b/src/com/android/launcher2/LauncherModel.java
index c6813c378..72675b94d 100644
--- a/src/com/android/launcher2/LauncherModel.java
+++ b/src/com/android/launcher2/LauncherModel.java
@@ -842,6 +842,8 @@ public class LauncherModel extends BroadcastReceiver {
                                 
                                 if (!isSafeMode && (provider == null || provider.provider == null ||
                                         provider.provider.getPackageName() == null)) {
+                                    Log.e(TAG, "Deleting widget that isn't installed anymore: id="
+                                            + id + " appWidgetId=" + appWidgetId);
                                     itemsToRemove.add(id);
                                 } else {
                                     appWidgetInfo = new LauncherAppWidgetInfo(appWidgetId);
@@ -1114,6 +1116,9 @@ public class LauncherModel extends BroadcastReceiver {
             return null;
         }
 
+        // TODO: See if the PackageManager knows about this case.  If it doesn't
+        // then return null & delete this.
+
         // the resource -- This may implicitly give us back the fallback icon,
         // but don't worry about that.  All we're doing with usingFallbackIcon is
         // to avoid saving lots of copies of that in the database, and most apps
@@ -1164,6 +1169,8 @@ public class LauncherModel extends BroadcastReceiver {
         final ShortcutInfo info = new ShortcutInfo();
         info.itemType = LauncherSettings.Favorites.ITEM_TYPE_SHORTCUT;
 
+        // TODO: If there's an explicit component and we can't install that, delete it.
+
         info.title = c.getString(titleIndex);
 
         int iconType = c.getInt(iconTypeIndex);