diff options
| author | Ray Chen <raychen@google.com> | 2011-09-09 11:20:58 +0800 |
|---|---|---|
| committer | Ray Chen <raychen@google.com> | 2011-09-09 11:31:25 +0800 |
| commit | 93db577592f48c09b87865db43bab4e93a6ab732 (patch) | |
| tree | b4766e1ab11a7c7ad50b3e1e2db404214ca1dbe9 /src/com/android/gallery3d/app | |
| parent | 6ee3afc7db3f844b92f5ad8c14cf7349167f7000 (diff) | |
| download | android_packages_apps_Snap-93db577592f48c09b87865db43bab4e93a6ab732.tar.gz android_packages_apps_Snap-93db577592f48c09b87865db43bab4e93a6ab732.tar.bz2 android_packages_apps_Snap-93db577592f48c09b87865db43bab4e93a6ab732.zip | |
Fix 5267234 com.google.android.gallery3d: java.lang.IllegalArgumentException:
The stateview to be finished is not at the top of the stack:
setNavigationMode triggers recursive tabs onSelected events.
Change-Id: Id9f58b4a73c4d7836b62e33e2f9b3676ba3f95c2
Diffstat (limited to 'src/com/android/gallery3d/app')
| -rw-r--r-- | src/com/android/gallery3d/app/GalleryActionBar.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/com/android/gallery3d/app/GalleryActionBar.java b/src/com/android/gallery3d/app/GalleryActionBar.java index efa5fa5e8..3df5e91e7 100644 --- a/src/com/android/gallery3d/app/GalleryActionBar.java +++ b/src/com/android/gallery3d/app/GalleryActionBar.java @@ -159,13 +159,16 @@ public class GalleryActionBar implements ActionBar.TabListener { public void showClusterTabs(ClusterRunner runner) { Log.v(TAG, "showClusterTabs: runner=" + runner); + // setNavigationMode will trigger onTabSelected, so we should avoid + // triggering any callback here + mClusterRunner = null; mActionBar.setNavigationMode(ActionBar.NAVIGATION_MODE_TABS); mClusterRunner = runner; } public void hideClusterTabs() { - mActionBar.setNavigationMode(ActionBar.NAVIGATION_MODE_STANDARD); mClusterRunner = null; + mActionBar.setNavigationMode(ActionBar.NAVIGATION_MODE_STANDARD); Log.v(TAG, "hideClusterTabs: runner=" + mClusterRunner); } |