Destroy loader when done loading components

A strange bug was uncovered where items keep loading into the
selector which causes many duplicates.  Once we load the data
we can simply destroy the loader so we will not get any further
updates.

The root cause appears to be caused by the post processing of
themes in the ThemeService.  The ThemesProvider listens for changes
to theme resources and updates the preview blobs which triggers the
loaders to be triggered.

Change-Id: Ifbeddadd2cd47d0f04755036fbb432d0fafb3df2

1 files changed, 8 insertions, 1 deletions

diff --git a/src/com/cyngn/theme/chooser/ComponentSelector.java b/src/com/cyngn/theme/chooser/ComponentSelector.java
index 881f636..9450c4a 100644
--- a/src/com/cyngn/theme/chooser/ComponentSelector.java
+++ b/src/com/cyngn/theme/chooser/ComponentSelector.java
@@ -416,7 +416,6 @@ public class ComponentSelector extends LinearLayout
 
     @Override
     public void onLoaderReset(Loader<Cursor> loader) {
-        mContent.removeAllViews();
     }
 
     public void setOnItemClickedListener(OnItemClickedListener listener) {
@@ -670,6 +669,14 @@ public class ComponentSelector extends LinearLayout
                     }
                 });
             }
+            // destroy the loader now that we are done with it
+            ComponentSelector.this.post(new Runnable() {
+                @Override
+                public void run() {
+                    ((FragmentActivity)mContext).getSupportLoaderManager().destroyLoader(
+                            getLoaderIdFromComponent(mComponentType));
+                }
+            });
             return null;
         }
     }