From 3aaf14ef3f2ed0099903a2d42ee5578282b8d331 Mon Sep 17 00:00:00 2001
From: seven332 <seven332@163.com>
Date: Mon, 17 Aug 2015 21:57:00 +0800
Subject: [PATCH] Fix BadParcelableException error

---
 .../widget/recyclerview/EasyRecyclerView.java | 46 +++++++++++++++++--
 1 file changed, 42 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/com/hippo/widget/recyclerview/EasyRecyclerView.java b/app/src/main/java/com/hippo/widget/recyclerview/EasyRecyclerView.java
index 4b3b2bff8..774c3d0d5 100644
--- a/app/src/main/java/com/hippo/widget/recyclerview/EasyRecyclerView.java
+++ b/app/src/main/java/com/hippo/widget/recyclerview/EasyRecyclerView.java
@@ -977,24 +977,52 @@ public class EasyRecyclerView extends RecyclerView {
         setViewChecked(view, isItemChecked(position));
     }
 
-    static class SavedState extends BaseSavedState {
+    /**
+     * This saved state class is a Parcelable and should not extend
+     * {@link android.view.View.BaseSavedState} nor {@link android.view.AbsSavedState}
+     * because its super class AbsSavedState's constructor
+     * {@link android.view.AbsSavedState#AbsSavedState(Parcel)} currently passes null
+     * as a class loader to read its superstate from Parcelable.
+     * This causes {@link android.os.BadParcelableException} when restoring saved states.
+     * <p/>
+     * The super class "RecyclerView" is a part of the support library,
+     * and restoring its saved state requires the class loader that loaded the RecyclerView.
+     * It seems that the class loader is not required when restoring from RecyclerView itself,
+     * but it is required when restoring from RecyclerView's subclasses.
+     */
+    static class SavedState implements Parcelable {
+
+        public static final SavedState EMPTY_STATE = new SavedState() {
+        };
+
         int choiceMode;
         int checkedItemCount;
         SparseBooleanArray checkState;
         LongSparseArray<Integer> checkIdState;
 
+        // This keeps the parent(RecyclerView)'s state
+        Parcelable mSuperState;
+
+        SavedState() {
+            mSuperState = null;
+        }
+
         /**
          * Constructor called from {@link #onSaveInstanceState()}
          */
         SavedState(Parcelable superState) {
-            super(superState);
+            mSuperState = superState != EMPTY_STATE ? superState : null;
         }
 
         /**
          * Constructor called from {@link #CREATOR}
          */
         private SavedState(Parcel in) {
-            super(in);
+            // Parcel 'in' has its parent(RecyclerView)'s saved state.
+            // To restore it, class loader that loaded RecyclerView is required.
+            Parcelable superState = in.readParcelable(RecyclerView.class.getClassLoader());
+            mSuperState = superState != null ? superState : EMPTY_STATE;
+
             choiceMode = in.readInt();
             checkedItemCount = in.readInt();
             checkState = in.readSparseBooleanArray();
@@ -1009,9 +1037,15 @@ public class EasyRecyclerView extends RecyclerView {
             }
         }
 
+        @Override
+        public int describeContents() {
+            return 0;
+        }
+
         @Override
         public void writeToParcel(@NonNull Parcel out, int flags) {
-            super.writeToParcel(out, flags);
+            out.writeParcelable(mSuperState, flags);
+
             out.writeInt(choiceMode);
             out.writeInt(checkedItemCount);
             out.writeSparseBooleanArray(checkState);
@@ -1023,6 +1057,10 @@ public class EasyRecyclerView extends RecyclerView {
             }
         }
 
+        public Parcelable getSuperState() {
+            return mSuperState;
+        }
+
         public static final Parcelable.Creator<SavedState> CREATOR
                 = new Parcelable.Creator<SavedState>() {
             @Override