Trebuchet: Prevent crashes on view id mess up

Same fix as: https://github.com/CyanogenMod/android_packages_apps_Trebuchet/commit/33a152fa

Issue-id: CYAN-7243

Change-Id: I558ef03b0299cefc3efd5e51f2ca6542836e66ff

1 files changed, 15 insertions, 3 deletions

diff --git a/src/com/android/launcher3/Launcher.java b/src/com/android/launcher3/Launcher.java
index bada6ba13..030e2712d 100644
--- a/src/com/android/launcher3/Launcher.java
+++ b/src/com/android/launcher3/Launcher.java
@@ -2357,9 +2357,21 @@ public class Launcher extends Activity
 
     @Override
     public void onRestoreInstanceState(Bundle state) {
-        super.onRestoreInstanceState(state);
-        for (int page: mSynchronouslyBoundPages) {
-            mWorkspace.restoreInstanceStateForChild(page);
+        /*
+         *  Prevent crashes on view id mess up
+         *  See: https://github.com/CyanogenMod/android_packages_apps_Trebuchet/commit/33a152fa
+         */
+        try {
+            super.onRestoreInstanceState(state);
+            for (int page: mSynchronouslyBoundPages) {
+                mWorkspace.restoreInstanceStateForChild(page);
+            }
+        } catch (IllegalArgumentException ex) {
+            if (LauncherAppState.isDogfoodBuild()) {
+                throw ex;
+            }
+            // Mismatched viewId / viewType preventing restore. Skip restore on production builds.
+            Log.e(TAG, "Ignoring an error while restoring a view instance state", ex);
         }
     }