diff options
author | Mangesh Ghiware <mghiware@google.com> | 2012-09-28 13:48:13 -0700 |
---|---|---|
committer | Mangesh Ghiware <mghiware@google.com> | 2012-09-28 13:48:13 -0700 |
commit | 9055cd7d1fecd9b73d1907cac9bc234cb8021894 (patch) | |
tree | f46dd5e11aa36839f889b56751729fb1c92ad891 /src | |
parent | 422dd98663e78de1b16accfde49cabda8d5a3568 (diff) | |
download | android_packages_apps_Snap-9055cd7d1fecd9b73d1907cac9bc234cb8021894.tar.gz android_packages_apps_Snap-9055cd7d1fecd9b73d1907cac9bc234cb8021894.tar.bz2 android_packages_apps_Snap-9055cd7d1fecd9b73d1907cac9bc234cb8021894.zip |
Fix NPE in GalleryActionBar
Bug: 7252666
Change-Id: Ide3c8e0d609743b34a3c53f3bb48dbeddb3f98c0
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/gallery3d/app/GalleryActionBar.java | 28 | ||||
-rw-r--r-- | src/com/android/gallery3d/ui/ActionModeHandler.java | 2 |
2 files changed, 17 insertions, 13 deletions
diff --git a/src/com/android/gallery3d/app/GalleryActionBar.java b/src/com/android/gallery3d/app/GalleryActionBar.java index 867ca6b3b..2630c3e4a 100644 --- a/src/com/android/gallery3d/app/GalleryActionBar.java +++ b/src/com/android/gallery3d/app/GalleryActionBar.java @@ -324,18 +324,22 @@ public class GalleryActionBar implements OnNavigationListener { public void setShareIntents(Intent sharePanoramaIntent, Intent shareIntent) { // if panorama sharing is enabled, rename share to share as photo, // and move it to overflow - if (sharePanoramaIntent != null) { - mActivity.invalidateOptionsMenu(); - mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER); - mShareMenuItem.setTitle( - mContext.getResources().getString(R.string.share_as_photo)); - } else { - mSharePanoramaMenuItem.setVisible(false); - mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM); - mShareMenuItem.setTitle( - mContext.getResources().getString(R.string.share)); + if (mSharePanoramaMenuItem != null) { + if (sharePanoramaIntent != null) { + mActivity.invalidateOptionsMenu(); + mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER); + mShareMenuItem.setTitle( + mContext.getResources().getString(R.string.share_as_photo)); + } else { + mSharePanoramaMenuItem.setVisible(false); + mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM); + mShareMenuItem.setTitle( + mContext.getResources().getString(R.string.share)); + } + mSharePanoramaActionProvider.setShareIntent(sharePanoramaIntent); + } + if (mShareMenuItem != null) { + mShareActionProvider.setShareIntent(shareIntent); } - mSharePanoramaActionProvider.setShareIntent(sharePanoramaIntent); - mShareActionProvider.setShareIntent(shareIntent); } } diff --git a/src/com/android/gallery3d/ui/ActionModeHandler.java b/src/com/android/gallery3d/ui/ActionModeHandler.java index f0ddde839..e4b3b4125 100644 --- a/src/com/android/gallery3d/ui/ActionModeHandler.java +++ b/src/com/android/gallery3d/ui/ActionModeHandler.java @@ -358,7 +358,7 @@ public class ActionModeHandler implements Callback, PopupList.OnPopupItemClickLi mSharePanoramaMenuItem.setEnabled(true); if ((operation & MediaObject.SUPPORT_PANORAMA) != 0) { mActivity.invalidateOptionsMenu(); - mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM); + mShareMenuItem.setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER); mShareMenuItem.setTitle( mActivity.getResources().getString(R.string.share_as_photo)); } else { |