diff options
author | Bjorn Bringert <bringert@google.com> | 2013-09-13 13:39:12 +0100 |
---|---|---|
committer | Bjorn Bringert <bringert@google.com> | 2013-09-13 13:42:00 +0100 |
commit | c639f8cdab11c603e25b5ae38282dddcb939d401 (patch) | |
tree | 6a84ff2120080b1a0311193b0569070717a1fa0d /src | |
parent | 80521d95f96f6eb630c269f899ecc28b809fb5ba (diff) | |
download | android_packages_apps_Trebuchet-c639f8cdab11c603e25b5ae38282dddcb939d401.tar.gz android_packages_apps_Trebuchet-c639f8cdab11c603e25b5ae38282dddcb939d401.tar.bz2 android_packages_apps_Trebuchet-c639f8cdab11c603e25b5ae38282dddcb939d401.zip |
Don't use resource ID in switch
This makes Eclipse unhappy when the code is used
as a library project.
Change-Id: I75949ebd36fabcdc88914150dc3a21aecc3357d8
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/launcher3/WallpaperPickerActivity.java | 40 |
1 files changed, 20 insertions, 20 deletions
diff --git a/src/com/android/launcher3/WallpaperPickerActivity.java b/src/com/android/launcher3/WallpaperPickerActivity.java index a0b9e6e7d..fe7525a00 100644 --- a/src/com/android/launcher3/WallpaperPickerActivity.java +++ b/src/com/android/launcher3/WallpaperPickerActivity.java @@ -306,27 +306,27 @@ public class WallpaperPickerActivity extends WallpaperCropActivity { // Called when the user selects a contextual menu item @Override public boolean onActionItemClicked(ActionMode mode, MenuItem item) { - switch (item.getItemId()) { - case R.id.menu_delete: - int childCount = mWallpapersView.getChildCount(); - ArrayList<View> viewsToRemove = new ArrayList<View>(); - for (int i = 0; i < childCount; i++) { - CheckableFrameLayout c = - (CheckableFrameLayout) mWallpapersView.getChildAt(i); - if (c.isChecked()) { - ThumbnailMetaData meta = (ThumbnailMetaData) c.getTag(); - mSavedImages.deleteImage(meta.mSavedWallpaperDbId); - viewsToRemove.add(c); - } - } - for (View v : viewsToRemove) { - mWallpapersView.removeView(v); + int itemId = item.getItemId(); + if (itemId == R.id.menu_delete) { + int childCount = mWallpapersView.getChildCount(); + ArrayList<View> viewsToRemove = new ArrayList<View>(); + for (int i = 0; i < childCount; i++) { + CheckableFrameLayout c = + (CheckableFrameLayout) mWallpapersView.getChildAt(i); + if (c.isChecked()) { + ThumbnailMetaData meta = (ThumbnailMetaData) c.getTag(); + mSavedImages.deleteImage(meta.mSavedWallpaperDbId); + viewsToRemove.add(c); } - ///xxxxx DESTROYING - mode.finish(); // Action picked, so close the CAB - return true; - default: - return false; + } + for (View v : viewsToRemove) { + mWallpapersView.removeView(v); + } + ///xxxxx DESTROYING + mode.finish(); // Action picked, so close the CAB + return true; + } else { + return false; } } |