diff options
author | Michael Kolb <kolby@google.com> | 2013-04-12 17:00:38 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2013-04-12 17:00:38 +0000 |
commit | 743dfdcd88081ff19fa6100dbd7d9c84aabe71c0 (patch) | |
tree | 9793f8676cb7e01a3322ae177c4de36d93b3c1a6 /src/com/android/camera | |
parent | 6608b57af0fd3b561022b589d4dc7ece567cf759 (diff) | |
parent | 4205d50cbb7fcf822e992c955719d4f8872f37d5 (diff) | |
download | android_packages_apps_Snap-743dfdcd88081ff19fa6100dbd7d9c84aabe71c0.tar.gz android_packages_apps_Snap-743dfdcd88081ff19fa6100dbd7d9c84aabe71c0.tar.bz2 android_packages_apps_Snap-743dfdcd88081ff19fa6100dbd7d9c84aabe71c0.zip |
Merge "Add checks for NPE in video pie menu." into gb-ub-photos-bryce
Diffstat (limited to 'src/com/android/camera')
-rw-r--r-- | src/com/android/camera/VideoMenu.java | 53 |
1 files changed, 31 insertions, 22 deletions
diff --git a/src/com/android/camera/VideoMenu.java b/src/com/android/camera/VideoMenu.java index c9f293222..ea70ddcdd 100644 --- a/src/com/android/camera/VideoMenu.java +++ b/src/com/android/camera/VideoMenu.java @@ -59,30 +59,39 @@ public class VideoMenu extends PieController super.initialize(group); mPopup = null; mPopupStatus = POPUP_NONE; - - PieItem item = makeItem(CameraSettings.KEY_VIDEOCAMERA_FLASH_MODE, POS_FLASH, 5); - mRenderer.addItem(item); - item = makeItem(CameraSettings.KEY_WHITE_BALANCE, POS_WB, 5); - mRenderer.addItem(item); + PieItem item = null; + // flash + if (group.findPreference(CameraSettings.KEY_VIDEOCAMERA_FLASH_MODE) != null) { + item = makeItem(CameraSettings.KEY_VIDEOCAMERA_FLASH_MODE, POS_FLASH, 5); + mRenderer.addItem(item); + } + // white balance + if (group.findPreference(CameraSettings.KEY_WHITE_BALANCE) != null) { + item = makeItem(CameraSettings.KEY_WHITE_BALANCE, POS_WB, 5); + mRenderer.addItem(item); + } // camera switcher - item = makeItem(R.drawable.ic_switch_video_facing_holo_light); - item.setPosition(POS_SWITCH, 5); - item.setOnClickListener(new OnClickListener() { - - @Override - public void onClick(PieItem item) { - // Find the index of next camera. - ListPreference pref = mPreferenceGroup.findPreference(CameraSettings.KEY_CAMERA_ID); - if (pref != null) { - int index = pref.findIndexOfValue(pref.getValue()); - CharSequence[] values = pref.getEntryValues(); - index = (index + 1) % values.length; - int newCameraId = Integer.parseInt((String) values[index]); - mListener.onCameraPickerClicked(newCameraId); + if (group.findPreference(CameraSettings.KEY_CAMERA_ID) != null) { + item = makeItem(R.drawable.ic_switch_video_facing_holo_light); + item.setPosition(POS_SWITCH, 5); + item.setOnClickListener(new OnClickListener() { + + @Override + public void onClick(PieItem item) { + // Find the index of next camera. + ListPreference pref = + mPreferenceGroup.findPreference(CameraSettings.KEY_CAMERA_ID); + if (pref != null) { + int index = pref.findIndexOfValue(pref.getValue()); + CharSequence[] values = pref.getEntryValues(); + index = (index + 1) % values.length; + int newCameraId = Integer.parseInt((String) values[index]); + mListener.onCameraPickerClicked(newCameraId); + } } - } - }); - mRenderer.addItem(item); + }); + mRenderer.addItem(item); + } // settings popup mOtherKeys = new String[] { CameraSettings.KEY_VIDEO_EFFECT, |