Fix Back behavior in gallery grid and remove camera preview tile.

Bug: 6411801
Bug: 6411709

Change-Id: Iebd7db06d44d315ac67eb8c5133b86044fce30c4

1 files changed, 18 insertions, 13 deletions

diff --git a/src/com/android/gallery3d/app/PhotoPage.java b/src/com/android/gallery3d/app/PhotoPage.java
index afa56c905..a237cbac3 100644
--- a/src/com/android/gallery3d/app/PhotoPage.java
+++ b/src/com/android/gallery3d/app/PhotoPage.java
@@ -110,6 +110,8 @@ public class PhotoPage extends ActivityState implements
     private boolean mIsActive;
     private ShareActionProvider mShareActionProvider;
     private String mSetPathString;
+    // This is the original mSetPathString before adding the camera preview item.
+    private String mOriginalSetPathString;
     private AppBridge mAppBridge;
     private ScreenNail mScreenNail;
     private MediaItem mScreenNailItem;
@@ -171,6 +173,7 @@ public class PhotoPage extends ActivityState implements
         mOrientationManager.addListener(this);
 
         mSetPathString = data.getString(KEY_MEDIA_SET_PATH);
+        mOriginalSetPathString = mSetPathString;
         mNfcAdapter = NfcAdapter.getDefaultAdapter(mActivity.getAndroidContext());
         Path itemPath = Path.fromString(data.getString(KEY_MEDIA_ITEM_PATH));
 
@@ -427,6 +430,20 @@ public class PhotoPage extends ActivityState implements
         }
     }
 
+    private void onUpPressed() {
+        if (mActivity.getStateManager().getStateCount() > 1) {
+            super.onBackPressed();
+        } else if (mOriginalSetPathString != null) {
+            // We're in view mode so set up the stacks on our own.
+            Bundle data = new Bundle(getData());
+            data.putString(AlbumPage.KEY_MEDIA_PATH, mOriginalSetPathString);
+            data.putString(AlbumPage.KEY_PARENT_MEDIA_PATH,
+                    mActivity.getDataManager().getTopSetPath(
+                            DataManager.INCLUDE_ALL));
+            mActivity.getStateManager().switchState(this, AlbumPage.class, data);
+        }
+    }
+
     private void setResult() {
         Intent result = null;
         if (!mPhotoView.getFilmMode()) {
@@ -480,19 +497,7 @@ public class PhotoPage extends ActivityState implements
         boolean needsConfirm = false;
         switch (action) {
             case android.R.id.home: {
-                if (mSetPathString != null) {
-                    if (mActivity.getStateManager().getStateCount() > 1) {
-                        onBackPressed();
-                    } else {
-                        // We're in view mode so set up the stacks on our own.
-                        Bundle data = new Bundle(getData());
-                        data.putString(AlbumPage.KEY_MEDIA_PATH, mSetPathString);
-                        data.putString(AlbumPage.KEY_PARENT_MEDIA_PATH,
-                                mActivity.getDataManager().getTopSetPath(
-                                        DataManager.INCLUDE_ALL));
-                        mActivity.getStateManager().switchState(this, AlbumPage.class, data);
-                    }
-                }
+                onUpPressed();
                 return true;
             }
             case R.id.action_slideshow: {