diff options
author | Doris Liu <tianliu@google.com> | 2014-03-27 14:43:31 -0700 |
---|---|---|
committer | Doris Liu <tianliu@google.com> | 2014-03-28 12:16:30 -0700 |
commit | 8ad8ad4841f65ee89e3c5e79f57d95cf8e4438b4 (patch) | |
tree | d7de311010faea28b3f579aae50439b88f41dcda | |
parent | b6db6917db367e4ca0992ca8614a4cfc59d68d92 (diff) | |
download | android_packages_apps_Camera2-8ad8ad4841f65ee89e3c5e79f57d95cf8e4438b4.tar.gz android_packages_apps_Camera2-8ad8ad4841f65ee89e3c5e79f57d95cf8e4438b4.tar.bz2 android_packages_apps_Camera2-8ad8ad4841f65ee89e3c5e79f57d95cf8e4438b4.zip |
Remove white balance and non-HDR scene mode settings
Also hook up HDR setting with scene mode to ensure it functions correctly
Bug: 13546569
Bug: 13546102
Change-Id: I71ef1841fc2bcb22f6ca0e286c31806ea049c79e
59 files changed, 146 insertions, 526 deletions
diff --git a/res/drawable-hdpi/ic_indicator_wb_cloudy.png b/res/drawable-hdpi/ic_indicator_wb_cloudy.png Binary files differdeleted file mode 100644 index 62f5c9985..000000000 --- a/res/drawable-hdpi/ic_indicator_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_indicator_wb_daylight.png b/res/drawable-hdpi/ic_indicator_wb_daylight.png Binary files differdeleted file mode 100644 index ffadb0e84..000000000 --- a/res/drawable-hdpi/ic_indicator_wb_daylight.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_indicator_wb_fluorescent.png b/res/drawable-hdpi/ic_indicator_wb_fluorescent.png Binary files differdeleted file mode 100644 index 3c30bfb73..000000000 --- a/res/drawable-hdpi/ic_indicator_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_indicator_wb_off.png b/res/drawable-hdpi/ic_indicator_wb_off.png Binary files differdeleted file mode 100644 index abe7405e1..000000000 --- a/res/drawable-hdpi/ic_indicator_wb_off.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_indicator_wb_tungsten.png b/res/drawable-hdpi/ic_indicator_wb_tungsten.png Binary files differdeleted file mode 100644 index 6de19d56d..000000000 --- a/res/drawable-hdpi/ic_indicator_wb_tungsten.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce.png b/res/drawable-hdpi/ic_sce.png Binary files differdeleted file mode 100644 index dd8bbd795..000000000 --- a/res/drawable-hdpi/ic_sce.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce_action.png b/res/drawable-hdpi/ic_sce_action.png Binary files differdeleted file mode 100644 index f96859291..000000000 --- a/res/drawable-hdpi/ic_sce_action.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce_night.png b/res/drawable-hdpi/ic_sce_night.png Binary files differdeleted file mode 100644 index dc42ec972..000000000 --- a/res/drawable-hdpi/ic_sce_night.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce_off.png b/res/drawable-hdpi/ic_sce_off.png Binary files differdeleted file mode 100644 index 4c64578a5..000000000 --- a/res/drawable-hdpi/ic_sce_off.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce_party.png b/res/drawable-hdpi/ic_sce_party.png Binary files differdeleted file mode 100644 index 6c664ba21..000000000 --- a/res/drawable-hdpi/ic_sce_party.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_sce_sunset.png b/res/drawable-hdpi/ic_sce_sunset.png Binary files differdeleted file mode 100644 index 82f74e6fb..000000000 --- a/res/drawable-hdpi/ic_sce_sunset.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_wb_auto.png b/res/drawable-hdpi/ic_wb_auto.png Binary files differdeleted file mode 100644 index 0e07a041f..000000000 --- a/res/drawable-hdpi/ic_wb_auto.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_wb_cloudy.png b/res/drawable-hdpi/ic_wb_cloudy.png Binary files differdeleted file mode 100644 index fb42ada92..000000000 --- a/res/drawable-hdpi/ic_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_wb_fluorescent.png b/res/drawable-hdpi/ic_wb_fluorescent.png Binary files differdeleted file mode 100644 index fbf8c9682..000000000 --- a/res/drawable-hdpi/ic_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_wb_incandescent.png b/res/drawable-hdpi/ic_wb_incandescent.png Binary files differdeleted file mode 100644 index 295a807e6..000000000 --- a/res/drawable-hdpi/ic_wb_incandescent.png +++ /dev/null diff --git a/res/drawable-hdpi/ic_wb_sunlight.png b/res/drawable-hdpi/ic_wb_sunlight.png Binary files differdeleted file mode 100644 index d256509be..000000000 --- a/res/drawable-hdpi/ic_wb_sunlight.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_indicator_wb_cloudy.png b/res/drawable-mdpi/ic_indicator_wb_cloudy.png Binary files differdeleted file mode 100644 index 858394fb3..000000000 --- a/res/drawable-mdpi/ic_indicator_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_indicator_wb_daylight.png b/res/drawable-mdpi/ic_indicator_wb_daylight.png Binary files differdeleted file mode 100644 index b8520e1ca..000000000 --- a/res/drawable-mdpi/ic_indicator_wb_daylight.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_indicator_wb_fluorescent.png b/res/drawable-mdpi/ic_indicator_wb_fluorescent.png Binary files differdeleted file mode 100644 index cb34af186..000000000 --- a/res/drawable-mdpi/ic_indicator_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_indicator_wb_off.png b/res/drawable-mdpi/ic_indicator_wb_off.png Binary files differdeleted file mode 100644 index f20ddf50e..000000000 --- a/res/drawable-mdpi/ic_indicator_wb_off.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_indicator_wb_tungsten.png b/res/drawable-mdpi/ic_indicator_wb_tungsten.png Binary files differdeleted file mode 100644 index ebfb22575..000000000 --- a/res/drawable-mdpi/ic_indicator_wb_tungsten.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce.png b/res/drawable-mdpi/ic_sce.png Binary files differdeleted file mode 100644 index 724d27820..000000000 --- a/res/drawable-mdpi/ic_sce.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce_action.png b/res/drawable-mdpi/ic_sce_action.png Binary files differdeleted file mode 100644 index 2cd9a8c7c..000000000 --- a/res/drawable-mdpi/ic_sce_action.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce_night.png b/res/drawable-mdpi/ic_sce_night.png Binary files differdeleted file mode 100644 index 94792623f..000000000 --- a/res/drawable-mdpi/ic_sce_night.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce_off.png b/res/drawable-mdpi/ic_sce_off.png Binary files differdeleted file mode 100644 index 2a3f9503f..000000000 --- a/res/drawable-mdpi/ic_sce_off.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce_party.png b/res/drawable-mdpi/ic_sce_party.png Binary files differdeleted file mode 100644 index 5f0f9cf1e..000000000 --- a/res/drawable-mdpi/ic_sce_party.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_sce_sunset.png b/res/drawable-mdpi/ic_sce_sunset.png Binary files differdeleted file mode 100644 index 0cede836c..000000000 --- a/res/drawable-mdpi/ic_sce_sunset.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_wb_auto.png b/res/drawable-mdpi/ic_wb_auto.png Binary files differdeleted file mode 100644 index 3da3a5c0b..000000000 --- a/res/drawable-mdpi/ic_wb_auto.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_wb_cloudy.png b/res/drawable-mdpi/ic_wb_cloudy.png Binary files differdeleted file mode 100644 index 5dc6fbd08..000000000 --- a/res/drawable-mdpi/ic_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_wb_fluorescent.png b/res/drawable-mdpi/ic_wb_fluorescent.png Binary files differdeleted file mode 100644 index 87c001bc7..000000000 --- a/res/drawable-mdpi/ic_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_wb_incandescent.png b/res/drawable-mdpi/ic_wb_incandescent.png Binary files differdeleted file mode 100644 index f6512efff..000000000 --- a/res/drawable-mdpi/ic_wb_incandescent.png +++ /dev/null diff --git a/res/drawable-mdpi/ic_wb_sunlight.png b/res/drawable-mdpi/ic_wb_sunlight.png Binary files differdeleted file mode 100644 index 4a07ec46c..000000000 --- a/res/drawable-mdpi/ic_wb_sunlight.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_indicator_wb_cloudy.png b/res/drawable-xhdpi/ic_indicator_wb_cloudy.png Binary files differdeleted file mode 100644 index 862ad5fce..000000000 --- a/res/drawable-xhdpi/ic_indicator_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_indicator_wb_daylight.png b/res/drawable-xhdpi/ic_indicator_wb_daylight.png Binary files differdeleted file mode 100644 index 5c2a4ddf8..000000000 --- a/res/drawable-xhdpi/ic_indicator_wb_daylight.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_indicator_wb_fluorescent.png b/res/drawable-xhdpi/ic_indicator_wb_fluorescent.png Binary files differdeleted file mode 100644 index 226b36cf8..000000000 --- a/res/drawable-xhdpi/ic_indicator_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_indicator_wb_off.png b/res/drawable-xhdpi/ic_indicator_wb_off.png Binary files differdeleted file mode 100644 index c5e38b1b4..000000000 --- a/res/drawable-xhdpi/ic_indicator_wb_off.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_indicator_wb_tungsten.png b/res/drawable-xhdpi/ic_indicator_wb_tungsten.png Binary files differdeleted file mode 100644 index a0b448585..000000000 --- a/res/drawable-xhdpi/ic_indicator_wb_tungsten.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce.png b/res/drawable-xhdpi/ic_sce.png Binary files differdeleted file mode 100644 index 44d0d7a0e..000000000 --- a/res/drawable-xhdpi/ic_sce.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce_action.png b/res/drawable-xhdpi/ic_sce_action.png Binary files differdeleted file mode 100644 index d79a7e0c1..000000000 --- a/res/drawable-xhdpi/ic_sce_action.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce_night.png b/res/drawable-xhdpi/ic_sce_night.png Binary files differdeleted file mode 100644 index 5dd62fcc6..000000000 --- a/res/drawable-xhdpi/ic_sce_night.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce_off.png b/res/drawable-xhdpi/ic_sce_off.png Binary files differdeleted file mode 100644 index 0279dcc10..000000000 --- a/res/drawable-xhdpi/ic_sce_off.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce_party.png b/res/drawable-xhdpi/ic_sce_party.png Binary files differdeleted file mode 100644 index 638d2d34d..000000000 --- a/res/drawable-xhdpi/ic_sce_party.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_sce_sunset.png b/res/drawable-xhdpi/ic_sce_sunset.png Binary files differdeleted file mode 100644 index 721966e43..000000000 --- a/res/drawable-xhdpi/ic_sce_sunset.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_wb_auto.png b/res/drawable-xhdpi/ic_wb_auto.png Binary files differdeleted file mode 100644 index cbacd04c4..000000000 --- a/res/drawable-xhdpi/ic_wb_auto.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_wb_cloudy.png b/res/drawable-xhdpi/ic_wb_cloudy.png Binary files differdeleted file mode 100644 index cf8ec8cd4..000000000 --- a/res/drawable-xhdpi/ic_wb_cloudy.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_wb_fluorescent.png b/res/drawable-xhdpi/ic_wb_fluorescent.png Binary files differdeleted file mode 100644 index 6e03a85e6..000000000 --- a/res/drawable-xhdpi/ic_wb_fluorescent.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_wb_incandescent.png b/res/drawable-xhdpi/ic_wb_incandescent.png Binary files differdeleted file mode 100644 index 367ded31e..000000000 --- a/res/drawable-xhdpi/ic_wb_incandescent.png +++ /dev/null diff --git a/res/drawable-xhdpi/ic_wb_sunlight.png b/res/drawable-xhdpi/ic_wb_sunlight.png Binary files differdeleted file mode 100644 index 497c4bdcc..000000000 --- a/res/drawable-xhdpi/ic_wb_sunlight.png +++ /dev/null diff --git a/res/layout/menu_indicators.xml b/res/layout/menu_indicators.xml deleted file mode 100644 index 037700349..000000000 --- a/res/layout/menu_indicators.xml +++ /dev/null @@ -1,57 +0,0 @@ -<?xml version="1.0" encoding="utf-8"?> -<!-- Copyright (C) 2013 The Android Open Source Project - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. ---> -<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android" - android:id="@+id/on_screen_indicators" - android:layout_width="64dip" - android:layout_height="64dip" > - - <ImageView - android:id="@+id/menu_scenemode_indicator" - style="@style/MenuIndicator" - android:layout_gravity="left|top" - android:src="@drawable/ic_indicator_sce_off" /> - - <ImageView - android:id="@+id/menu_timer_indicator" - style="@style/MenuIndicator" - android:layout_gravity="center_horizontal|top" - android:src="@drawable/ic_indicator_timer_off" /> - - <ImageView - android:id="@+id/menu_flash_indicator" - style="@style/MenuIndicator" - android:layout_gravity="right|top" - android:src="@drawable/ic_indicator_flash_off" /> - - <ImageView - android:id="@+id/menu_exposure_indicator" - style="@style/MenuIndicator" - android:layout_gravity="left|bottom" - android:src="@drawable/ic_indicator_ev_0" /> - - <ImageView - android:id="@+id/menu_location_indicator" - style="@style/MenuIndicator" - android:layout_gravity="center_horizontal|bottom" - android:src="@drawable/ic_indicator_loc_on" /> - - <ImageView - android:id="@+id/menu_wb_indicator" - style="@style/MenuIndicator" - android:layout_gravity="right|bottom" - android:src="@drawable/ic_indicator_wb_off" /> - -</FrameLayout> diff --git a/res/layout/menu_indicators_keyguard.xml b/res/layout/menu_indicators_keyguard.xml deleted file mode 100644 index f3631f67e..000000000 --- a/res/layout/menu_indicators_keyguard.xml +++ /dev/null @@ -1,57 +0,0 @@ -<?xml version="1.0" encoding="utf-8"?> -<!-- Copyright (C) 2012 The Android Open Source Project - - Licensed under the Apache License, Version 2.0 (the "License"); - you may not use this file except in compliance with the License. - You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - - Unless required by applicable law or agreed to in writing, software - distributed under the License is distributed on an "AS IS" BASIS, - WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - See the License for the specific language governing permissions and - limitations under the License. ---> -<FrameLayout xmlns:android="http://schemas.android.com/apk/res/android" - android:id="@+id/on_screen_indicators" - android:layout_width="64dip" - android:layout_height="64dip" > - - <ImageView - android:id="@+id/menu_scenemode_indicator" - style="@style/MenuIndicator" - android:layout_gravity="left|top" - android:src="@drawable/ic_indicator_sce_off" /> - - <ImageView - android:id="@+id/menu_timer_indicator" - style="@style/MenuIndicator" - android:layout_gravity="center_horizontal|top" - android:src="@drawable/ic_indicator_timer_off" /> - - <ImageView - android:id="@+id/menu_flash_indicator" - style="@style/MenuIndicator" - android:layout_gravity="right|top" - android:src="@drawable/ic_indicator_flash_off" /> - - <ImageView - android:id="@+id/menu_exposure_indicator" - style="@style/MenuIndicator" - android:layout_gravity="left|bottom" - android:src="@drawable/ic_indicator_ev_0" /> - - <ImageView - android:id="@+id/menu_location_indicator" - style="@style/MenuIndicator" - android:layout_gravity="center_horizontal|bottom" - android:src="@drawable/ic_indicator_loc_on" /> - - <ImageView - android:id="@+id/menu_wb_indicator" - style="@style/MenuIndicator" - android:layout_gravity="right|bottom" - android:src="@drawable/ic_indicator_wb_off" /> - -</FrameLayout> diff --git a/res/values/arrays.xml b/res/values/arrays.xml index c56e8cc3c..7f0d285cc 100644 --- a/res/values/arrays.xml +++ b/res/values/arrays.xml @@ -260,86 +260,9 @@ <item>@drawable/ic_flash_off_indicator</item> </array> - <!-- Camera Preferences White Balance dialog box entries --> - <string-array name="pref_camera_whitebalance_entries" translatable="false"> - <item>@string/pref_camera_whitebalance_entry_incandescent</item> - <item>@string/pref_camera_whitebalance_entry_fluorescent</item> - <item>@string/pref_camera_whitebalance_entry_auto</item> - <item>@string/pref_camera_whitebalance_entry_daylight</item> - <item>@string/pref_camera_whitebalance_entry_cloudy</item> - </string-array> - - <string-array name="pref_camera_whitebalance_labels" translatable="false"> - <item>@string/pref_camera_whitebalance_label_incandescent</item> - <item>@string/pref_camera_whitebalance_label_fluorescent</item> - <item>@string/pref_camera_whitebalance_label_auto</item> - <item>@string/pref_camera_whitebalance_label_daylight</item> - <item>@string/pref_camera_whitebalance_label_cloudy</item> - </string-array> - - <string-array name="pref_camera_whitebalance_entryvalues" translatable="false"> - <item>incandescent</item> - <item>fluorescent</item> - <item>auto</item> - <item>daylight</item> - <item>cloudy-daylight</item> - </string-array> - - <array name="whitebalance_icons" translatable="false"> - <item>@drawable/ic_wb_incandescent</item> - <item>@drawable/ic_wb_fluorescent</item> - <item>@drawable/ic_wb_auto</item> - <item>@drawable/ic_wb_sunlight</item> - <item>@drawable/ic_wb_cloudy</item> - </array> - - <array name="whitebalance_largeicons" translatable="false"> - <item>@drawable/ic_wb_incandescent</item> - <item>@drawable/ic_wb_fluorescent</item> - <item>@drawable/ic_wb_auto</item> - <item>@drawable/ic_wb_sunlight</item> - <item>@drawable/ic_wb_cloudy</item> - </array> - - <array name="camera_wb_indicators" translatable="false"> - <item>@drawable/ic_indicator_wb_tungsten</item> - <item>@drawable/ic_indicator_wb_fluorescent</item> - <item>@drawable/ic_indicator_wb_off</item> - <item>@drawable/ic_indicator_wb_daylight</item> - <item>@drawable/ic_indicator_wb_cloudy</item> - </array> - - <!-- Camera Preferences Scene Mode dialog box entries --> - <string-array name="pref_camera_scenemode_entries" translatable="false"> - <item>@string/pref_camera_scenemode_entry_action</item> - <item>@string/pref_camera_scenemode_entry_night</item> - <item>@string/pref_camera_scenemode_entry_auto</item> - <item>@string/pref_camera_scenemode_entry_sunset</item> - <item>@string/pref_camera_scenemode_entry_party</item> - </string-array> - - <string-array name="pref_camera_scenemode_labels"> - <item>@string/pref_camera_scenemode_label_action</item> - <item>@string/pref_camera_scenemode_label_night</item> - <item>@string/pref_camera_scenemode_label_auto</item> - <item>@string/pref_camera_scenemode_label_sunset</item> - <item>@string/pref_camera_scenemode_label_party</item> - </string-array> - - <array name="pref_camera_scenemode_icons"> - <item>@drawable/ic_sce_action</item> - <item>@drawable/ic_sce_night</item> - <item>@drawable/ic_sce_off</item> - <item>@drawable/ic_sce_sunset</item> - <item>@drawable/ic_sce_party</item> - </array> - <string-array name="pref_camera_scenemode_entryvalues" translatable="false"> - <item>action</item> - <item>night</item> <item>auto</item> - <item>sunset</item> - <item>party</item> + <item>hdr</item> </string-array> <array name="camera_id_entries" translatable="false"> diff --git a/src/com/android/camera/ButtonManager.java b/src/com/android/camera/ButtonManager.java index 66e28c15b..ddd3a51dd 100644 --- a/src/com/android/camera/ButtonManager.java +++ b/src/com/android/camera/ButtonManager.java @@ -166,11 +166,16 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { button = getButtonOrError(BUTTON_CAMERA); break; } - case SettingsManager.SETTING_CAMERA_HDR: { + case SettingsManager.SETTING_CAMERA_HDR_PLUS: { index = mSettingsManager.getStringValueIndex(id); button = getButtonOrError(BUTTON_HDRPLUS); break; } + case SettingsManager.SETTING_CAMERA_HDR: { + index = mSettingsManager.getStringValueIndex(id); + button = getButtonOrError(BUTTON_HDR); + break; + } case SettingsManager.SETTING_CAMERA_GRID_LINES: { index = mSettingsManager.getStringValueIndex(id); button = getButtonOrError(BUTTON_GRID_LINES); @@ -283,59 +288,47 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable a known button by id, with a state change callback and - * a resource id that points to an array of drawables. + * Initialize a known button by id, with a state change callback and + * a resource id that points to an array of drawables, and then enable + * the button. */ - public void enableButton(int buttonId, ButtonCallback cb) { + public void initializeButton(int buttonId, ButtonCallback cb) { MultiToggleImageButton button = getButtonOrError(buttonId); switch (buttonId) { case BUTTON_FLASH: - enableFlashButton(button, cb, R.array.camera_flashmode_icons); + initializeFlashButton(button, cb, R.array.camera_flashmode_icons); break; case BUTTON_TORCH: - enableTorchButton(button, cb, R.array.video_flashmode_icons); + initializeTorchButton(button, cb, R.array.video_flashmode_icons); break; case BUTTON_CAMERA: - enableCameraButton(button, cb, R.array.camera_id_icons); + initializeCameraButton(button, cb, R.array.camera_id_icons); break; case BUTTON_HDRPLUS: - enableHdrPlusButton(button, cb, R.array.pref_camera_hdr_plus_icons); + initializeHdrPlusButton(button, cb, R.array.pref_camera_hdr_plus_icons); break; case BUTTON_HDR: - enableHdrPlusButton(button, cb, R.array.pref_camera_hdr_icons); + initializeHdrButton(button, cb, R.array.pref_camera_hdr_icons); break; case BUTTON_GRID_LINES: - enableGridLinesButton(button, cb, R.array.grid_lines_icons); + initializeGridLinesButton(button, cb, R.array.grid_lines_icons); break; case BUTTON_PANO_ORIENTATION: - enablePanoOrientationButton(button, cb, - PhotoSphereHelper.getPanoramaOrientationOptionArrayId()); + initializePanoOrientationButton(button, cb, + PhotoSphereHelper.getPanoramaOrientationOptionArrayId()); break; default: throw new IllegalArgumentException("button not known by id=" + buttonId); } - if (!button.isEnabled()) { - button.setEnabled(true); - if (mListener != null) { - mListener.onButtonEnabledChanged(this, buttonId); - } - } - button.setTag(R.string.tag_enabled_id, (Integer) buttonId); - - if (button.getVisibility() != View.VISIBLE) { - button.setVisibility(View.VISIBLE); - if (mListener != null) { - mListener.onButtonVisibilityChanged(this, buttonId); - } - } + enableButton(buttonId); } /** - * Enable a known button with a click listener and a resource id. + * Initialize a known button with a click listener and a resource id. * Sets the button visible. */ - public void enablePushButton(int buttonId, View.OnClickListener cb, + public void initializePushButton(int buttonId, View.OnClickListener cb, int imageId) { ImageButton button = getImageButtonOrError(buttonId); button.setOnClickListener(cb); @@ -358,9 +351,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable a known button with a click listener. Sets the button visible. + * Initialize a known button with a click listener. Sets the button visible. */ - public void enablePushButton(int buttonId, View.OnClickListener cb) { + public void initializePushButton(int buttonId, View.OnClickListener cb) { ImageButton button = getImageButtonOrError(buttonId); button.setOnClickListener(cb); @@ -402,6 +395,27 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** + * Enables a button that has already been initialized. + */ + public void enableButton(int buttonId) { + ImageButton button = getButtonOrError(buttonId); + if (!button.isEnabled()) { + button.setEnabled(true); + if (mListener != null) { + mListener.onButtonEnabledChanged(this, buttonId); + } + } + button.setTag(R.string.tag_enabled_id, buttonId); + + if (button.getVisibility() != View.VISIBLE) { + button.setVisibility(View.VISIBLE); + if (mListener != null) { + mListener.onButtonVisibilityChanged(this, buttonId); + } + } + } + + /** * Hide a button by id. */ public void hideButton(int buttonId) { @@ -452,9 +466,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable a flash button. + * Initialize a flash button. */ - private void enableFlashButton(MultiToggleImageButton button, + private void initializeFlashButton(MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { @@ -477,9 +491,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable video torch button + * Initialize video torch button */ - private void enableTorchButton(MultiToggleImageButton button, + private void initializeTorchButton(MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { @@ -504,9 +518,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable a camera button. + * Initialize a camera button. */ - private void enableCameraButton(final MultiToggleImageButton button, + private void initializeCameraButton(final MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { @@ -536,9 +550,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable an hdr plus button. + * Initialize an hdr plus button. */ - private void enableHdrPlusButton(MultiToggleImageButton button, + private void initializeHdrPlusButton(MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { @@ -546,8 +560,34 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } button.overrideContentDescriptions(R.array.hdr_plus_descriptions); + int index = mSettingsManager.getStringValueIndex(SettingsManager.SETTING_CAMERA_HDR_PLUS); + button.setState(index >= 0 ? index : 0, false); + + button.setOnStateChangeListener(new MultiToggleImageButton.OnStateChangeListener() { + @Override + public void stateChanged(View view, int state) { + mSettingsManager.setStringValueIndex(SettingsManager.SETTING_CAMERA_HDR_PLUS, state); + if (cb != null) { + cb.onStateChanged(state); + } + } + }); + } + + /** + * Initialize an hdr button. + */ + private void initializeHdrButton(MultiToggleImageButton button, + final ButtonCallback cb, int resIdImages) { + + if (resIdImages > 0) { + button.overrideImageIds(resIdImages); + } + button.overrideContentDescriptions(R.array.hdr_descriptions); + int index = mSettingsManager.getStringValueIndex(SettingsManager.SETTING_CAMERA_HDR); button.setState(index >= 0 ? index : 0, false); + updateFlashButton(); button.setOnStateChangeListener(new MultiToggleImageButton.OnStateChangeListener() { @Override @@ -555,15 +595,29 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { mSettingsManager.setStringValueIndex(SettingsManager.SETTING_CAMERA_HDR, state); if (cb != null) { cb.onStateChanged(state); + updateFlashButton(); } } }); } - /** - * Enable a grid lines button. + + /** + * Flash button will be disabled (i.e. greyed out) when HDR is on. + */ + private void updateFlashButton() { + if (mSettingsManager.isHdrOn()) { + disableButton(BUTTON_FLASH); + } else { + enableButton(BUTTON_FLASH); + } + } + + + /** + * Initialize a grid lines button. */ - private void enableGridLinesButton(MultiToggleImageButton button, + private void initializeGridLinesButton(MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { @@ -587,9 +641,9 @@ public class ButtonManager implements SettingsManager.OnSettingChangedListener { } /** - * Enable a panorama orientation button. + * Initialize a panorama orientation button. */ - private void enablePanoOrientationButton(MultiToggleImageButton button, + private void initializePanoOrientationButton(MultiToggleImageButton button, final ButtonCallback cb, int resIdImages) { if (resIdImages > 0) { diff --git a/src/com/android/camera/OnScreenIndicators.java b/src/com/android/camera/OnScreenIndicators.java deleted file mode 100644 index a99b291d7..000000000 --- a/src/com/android/camera/OnScreenIndicators.java +++ /dev/null @@ -1,196 +0,0 @@ -/* - * Copyright (C) 2013 The Android Open Source Project - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * you may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package com.android.camera; - -import android.content.Context; -import android.content.res.TypedArray; -import android.hardware.Camera; -import android.hardware.Camera.Parameters; -import android.view.View; -import android.widget.ImageView; - -import com.android.camera2.R; - -/** - * The on-screen indicators of the pie menu button. They show the camera - * settings in the viewfinder. - */ -public class OnScreenIndicators { - public static final String SCENE_MODE_HDR_PLUS = "hdr_plus"; - - private final int[] mWBArray; - private final View mOnScreenIndicators; - private final ImageView mExposureIndicator; - private final ImageView mFlashIndicator; - private final ImageView mSceneIndicator; - private final ImageView mLocationIndicator; - private final ImageView mTimerIndicator; - private final ImageView mWBIndicator; - - public OnScreenIndicators(Context ctx, View onScreenIndicatorsView) { - TypedArray iconIds = ctx.getResources().obtainTypedArray( - R.array.camera_wb_indicators); - final int n = iconIds.length(); - mWBArray = new int[n]; - for (int i = 0; i < n; i++) { - mWBArray[i] = iconIds.getResourceId(i, R.drawable.ic_indicator_wb_off); - } - mOnScreenIndicators = onScreenIndicatorsView; - mExposureIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_exposure_indicator); - mFlashIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_flash_indicator); - mSceneIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_scenemode_indicator); - mLocationIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_location_indicator); - mTimerIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_timer_indicator); - mWBIndicator = (ImageView) onScreenIndicatorsView.findViewById( - R.id.menu_wb_indicator); - } - - /** - * Resets all indicators to show the default values. - */ - public void resetToDefault() { - updateExposureOnScreenIndicator(0); - updateFlashOnScreenIndicator(Parameters.FLASH_MODE_OFF); - updateSceneOnScreenIndicator(Parameters.SCENE_MODE_AUTO); - updateWBIndicator(2); - updateTimerIndicator(false); - updateLocationIndicator(false); - } - - /** - * Sets the exposure indicator using exposure compensations step rounding. - */ - public void updateExposureOnScreenIndicator(Camera.Parameters params, int value) { - if (mExposureIndicator == null) { - return; - } - float step = params.getExposureCompensationStep(); - value = Math.round(value * step); - updateExposureOnScreenIndicator(value); - } - - /** - * Set the exposure indicator to the given value. - * - * @param value Value between -3 and 3. If outside this range, 0 is used by - * default. - */ - public void updateExposureOnScreenIndicator(int value) { - int id = 0; - switch(value) { - case -3: - id = R.drawable.ic_indicator_ev_n3; - break; - case -2: - id = R.drawable.ic_indicator_ev_n2; - break; - case -1: - id = R.drawable.ic_indicator_ev_n1; - break; - case 0: - id = R.drawable.ic_indicator_ev_0; - break; - case 1: - id = R.drawable.ic_indicator_ev_p1; - break; - case 2: - id = R.drawable.ic_indicator_ev_p2; - break; - case 3: - id = R.drawable.ic_indicator_ev_p3; - break; - } - mExposureIndicator.setImageResource(id); - } - - public void updateWBIndicator(int wbIndex) { - if (mWBIndicator == null) return; - mWBIndicator.setImageResource(mWBArray[wbIndex]); - } - - public void updateTimerIndicator(boolean on) { - if (mTimerIndicator == null) return; - mTimerIndicator.setImageResource(on ? R.drawable.ic_indicator_timer_on - : R.drawable.ic_indicator_timer_off); - } - - public void updateLocationIndicator(boolean on) { - if (mLocationIndicator == null) return; - mLocationIndicator.setImageResource(on ? R.drawable.ic_indicator_loc_on - : R.drawable.ic_indicator_loc_off); - } - - /** - * Set the flash indicator to the given value. - * - * @param value One of Parameters.FLASH_MODE_OFF, - * Parameters.FLASH_MODE_AUTO, Parameters.FLASH_MODE_ON. - */ - public void updateFlashOnScreenIndicator(String value) { - if (mFlashIndicator == null) { - return; - } - if (value == null || Parameters.FLASH_MODE_OFF.equals(value)) { - mFlashIndicator.setImageResource(R.drawable.ic_indicator_flash_off); - } else { - if (Parameters.FLASH_MODE_AUTO.equals(value)) { - mFlashIndicator.setImageResource(R.drawable.ic_indicator_flash_auto); - } else if (Parameters.FLASH_MODE_ON.equals(value) - || Parameters.FLASH_MODE_TORCH.equals(value)) { - mFlashIndicator.setImageResource(R.drawable.ic_indicator_flash_on); - } else { - mFlashIndicator.setImageResource(R.drawable.ic_indicator_flash_off); - } - } - } - - /** - * Set the scene indicator depending on the given scene mode. - * - * @param value the current Parameters.SCENE_MODE_* value or - * {@link #SCENE_MODE_HDR_PLUS}. - */ - public void updateSceneOnScreenIndicator(String value) { - if (mSceneIndicator == null) { - return; - } - - if (SCENE_MODE_HDR_PLUS.equals(value)) { - mSceneIndicator.setImageResource(R.drawable.ic_indicator_hdr_plus_on); - } else if ((value == null) || Parameters.SCENE_MODE_AUTO.equals(value)) { - mSceneIndicator.setImageResource(R.drawable.ic_indicator_sce_off); - } else if (Parameters.SCENE_MODE_HDR.equals(value)) { - mSceneIndicator.setImageResource(R.drawable.ic_indicator_sce_hdr); - } else { - mSceneIndicator.setImageResource(R.drawable.ic_indicator_sce_on); - } - } - - /** - * Sets the visibility of all indicators. - * - * @param visibility View.VISIBLE, View.GONE etc. - */ - public void setVisibility(int visibility) { - mOnScreenIndicators.setVisibility(visibility); - } -} diff --git a/src/com/android/camera/PhotoModule.java b/src/com/android/camera/PhotoModule.java index 53f727b79..730c6af8d 100644 --- a/src/com/android/camera/PhotoModule.java +++ b/src/com/android/camera/PhotoModule.java @@ -308,7 +308,7 @@ public class PhotoModule private void switchToGcamCapture() { if (mActivity != null && mGcamModeIndex != 0) { SettingsManager settingsManager = mActivity.getSettingsManager(); - settingsManager.set(SettingsManager.SETTING_CAMERA_HDR, + settingsManager.set(SettingsManager.SETTING_CAMERA_HDR_PLUS, SettingsManager.VALUE_ON); // Disable the HDR+ button to prevent callbacks from being @@ -483,14 +483,22 @@ public class PhotoModule new ButtonManager.ButtonCallback() { @Override public void onStateChanged(int state) { + SettingsManager settingsManager = mActivity.getSettingsManager(); if (GcamHelper.hasGcamCapture()) { // Set the camera setting to default backfacing. - SettingsManager settingsManager = mActivity.getSettingsManager(); settingsManager.setDefault(SettingsManager.SETTING_CAMERA_ID); switchToGcamCapture(); } else { - mSceneMode = CameraUtil.SCENE_MODE_HDR; + if (settingsManager.isHdrOn()) { + settingsManager.set(SettingsManager.SETTING_SCENE_MODE, + CameraUtil.SCENE_MODE_HDR); + } else { + settingsManager.set(SettingsManager.SETTING_SCENE_MODE, + Parameters.SCENE_MODE_AUTO); + } updateParametersSceneMode(); + mCameraDevice.setParameters(mParameters); + updateSceneMode(); } } }; @@ -945,16 +953,13 @@ public class PhotoModule // If scene mode is set, we cannot set flash mode, white balance, and // focus mode, instead, we read it from driver if (!Parameters.SCENE_MODE_AUTO.equals(mSceneMode)) { - overrideCameraSettings(mParameters.getFlashMode(), - mParameters.getWhiteBalance(), mParameters.getFocusMode()); + overrideCameraSettings(mParameters.getFlashMode(), mParameters.getFocusMode()); } } - private void overrideCameraSettings(final String flashMode, - final String whiteBalance, final String focusMode) { + private void overrideCameraSettings(final String flashMode, final String focusMode) { SettingsManager settingsManager = mActivity.getSettingsManager(); settingsManager.set(SettingsManager.SETTING_FLASH_MODE, flashMode); - settingsManager.set(SettingsManager.SETTING_WHITE_BALANCE, whiteBalance); settingsManager.set(SettingsManager.SETTING_FOCUS_MODE, focusMode); } @@ -1695,9 +1700,6 @@ public class PhotoModule // Set flash mode. updateParametersFlashMode(); - // Set white balance mode. - updateParametersWhiteBalanceMode(); - // Set focus mode. mFocusManager.overrideFocusMode(null); mParameters.setFocusMode(mFocusManager.getFocusMode()); @@ -1716,17 +1718,6 @@ public class PhotoModule } } - private void updateParametersWhiteBalanceMode() { - SettingsManager settingsManager = mActivity.getSettingsManager(); - - // Set white balance parameter. - String whiteBalance = settingsManager.get(SettingsManager.SETTING_WHITE_BALANCE); - if (CameraUtil.isSupported(whiteBalance, - mParameters.getSupportedWhiteBalance())) { - mParameters.setWhiteBalance(whiteBalance); - } - } - @TargetApi(Build.VERSION_CODES.JELLY_BEAN) private void updateAutoFocusMoveCallback() { if (mParameters.getFocusMode().equals(CameraUtil.FOCUS_MODE_CONTINUOUS_PICTURE)) { diff --git a/src/com/android/camera/VideoModule.java b/src/com/android/camera/VideoModule.java index 309200974..0c8be23c9 100644 --- a/src/com/android/camera/VideoModule.java +++ b/src/com/android/camera/VideoModule.java @@ -1506,18 +1506,6 @@ public class VideoModule extends CameraModule enableTorchMode(settingsManager.isCameraBackFacing()); - // Set white balance parameter. - String whiteBalance = settingsManager.get(SettingsManager.SETTING_WHITE_BALANCE); - if (isSupported(whiteBalance, - mParameters.getSupportedWhiteBalance())) { - mParameters.setWhiteBalance(whiteBalance); - } else { - whiteBalance = mParameters.getWhiteBalance(); - if (whiteBalance == null) { - whiteBalance = Parameters.WHITE_BALANCE_AUTO; - } - } - // Set zoom. if (mParameters.isZoomSupported()) { mParameters.setZoom(mZoomValue); diff --git a/src/com/android/camera/app/CameraAppUI.java b/src/com/android/camera/app/CameraAppUI.java index cee03cff0..bd58816e2 100644 --- a/src/com/android/camera/app/CameraAppUI.java +++ b/src/com/android/camera/app/CameraAppUI.java @@ -1527,8 +1527,8 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, /** Standard mode options */ if (hardwareSpec.isFrontCameraSupported()) { if (bottomBarSpec.enableCamera) { - buttonManager.enableButton(ButtonManager.BUTTON_CAMERA, - bottomBarSpec.cameraCallback); + buttonManager.initializeButton(ButtonManager.BUTTON_CAMERA, + bottomBarSpec.cameraCallback); } else { buttonManager.disableButton(ButtonManager.BUTTON_CAMERA); } @@ -1544,9 +1544,9 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, } else { if (hardwareSpec.isFlashSupported()) { if (bottomBarSpec.enableFlash) { - buttonManager.enableButton(ButtonManager.BUTTON_FLASH, bottomBarSpec.flashCallback); + buttonManager.initializeButton(ButtonManager.BUTTON_FLASH, bottomBarSpec.flashCallback); } else if (bottomBarSpec.enableTorchFlash) { - buttonManager.enableButton(ButtonManager.BUTTON_TORCH, bottomBarSpec.flashCallback); + buttonManager.initializeButton(ButtonManager.BUTTON_TORCH, bottomBarSpec.flashCallback); } else { buttonManager.disableButton(ButtonManager.BUTTON_FLASH); } @@ -1562,15 +1562,15 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, } else { if (hardwareSpec.isHdrPlusSupported()) { if (bottomBarSpec.enableHdr && settingsManager.isCameraBackFacing()) { - buttonManager.enableButton(ButtonManager.BUTTON_HDRPLUS, - bottomBarSpec.hdrCallback); + buttonManager.initializeButton(ButtonManager.BUTTON_HDRPLUS, + bottomBarSpec.hdrCallback); } else { buttonManager.disableButton(ButtonManager.BUTTON_HDRPLUS); } } else if (hardwareSpec.isHdrSupported()) { if (bottomBarSpec.enableHdr && settingsManager.isCameraBackFacing()) { - buttonManager.enableButton(ButtonManager.BUTTON_HDR, - bottomBarSpec.hdrCallback); + buttonManager.initializeButton(ButtonManager.BUTTON_HDR, + bottomBarSpec.hdrCallback); } else { buttonManager.disableButton(ButtonManager.BUTTON_HDR); } @@ -1586,9 +1586,10 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, hideGridLines(); } else { if (bottomBarSpec.enableGridLines) { - buttonManager.enableButton(ButtonManager.BUTTON_GRID_LINES, - bottomBarSpec.gridLinesCallback != null ? - bottomBarSpec.gridLinesCallback : getGridLinesCallback()); + buttonManager.initializeButton(ButtonManager.BUTTON_GRID_LINES, + bottomBarSpec.gridLinesCallback != null ? + bottomBarSpec.gridLinesCallback : getGridLinesCallback() + ); } else { buttonManager.disableButton(ButtonManager.BUTTON_GRID_LINES); hideGridLines(); @@ -1597,7 +1598,7 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, if (bottomBarSpec.enablePanoOrientation && PhotoSphereHelper.getPanoramaOrientationOptionArrayId() > 0) { - buttonManager.enableButton(ButtonManager.BUTTON_PANO_ORIENTATION, + buttonManager.initializeButton(ButtonManager.BUTTON_PANO_ORIENTATION, bottomBarSpec.panoOrientationCallback); } else { buttonManager.hideButton(ButtonManager.BUTTON_PANO_ORIENTATION); @@ -1605,21 +1606,21 @@ public class CameraAppUI implements ModeListView.ModeSwitchListener, /** Intent UI */ if (bottomBarSpec.showCancel) { - buttonManager.enablePushButton(ButtonManager.BUTTON_CANCEL, - bottomBarSpec.cancelCallback); + buttonManager.initializePushButton(ButtonManager.BUTTON_CANCEL, + bottomBarSpec.cancelCallback); } if (bottomBarSpec.showDone) { - buttonManager.enablePushButton(ButtonManager.BUTTON_DONE, - bottomBarSpec.doneCallback); + buttonManager.initializePushButton(ButtonManager.BUTTON_DONE, + bottomBarSpec.doneCallback); } if (bottomBarSpec.showRetake) { - buttonManager.enablePushButton(ButtonManager.BUTTON_RETAKE, - bottomBarSpec.retakeCallback); + buttonManager.initializePushButton(ButtonManager.BUTTON_RETAKE, + bottomBarSpec.retakeCallback); } if (bottomBarSpec.showReview) { - buttonManager.enablePushButton(ButtonManager.BUTTON_REVIEW, - bottomBarSpec.reviewCallback, - R.drawable.ic_play); + buttonManager.initializePushButton(ButtonManager.BUTTON_REVIEW, + bottomBarSpec.reviewCallback, + R.drawable.ic_play); } } diff --git a/src/com/android/camera/settings/SettingsCache.java b/src/com/android/camera/settings/SettingsCache.java index 42df79424..7e01650b6 100644 --- a/src/com/android/camera/settings/SettingsCache.java +++ b/src/com/android/camera/settings/SettingsCache.java @@ -105,12 +105,8 @@ public class SettingsCache { mKeyMap.put(SettingsManager.KEY_FLASH_MODE, SettingsManager.SETTING_FLASH_MODE); mKeyMap.put(SettingsManager.KEY_VIDEOCAMERA_FLASH_MODE, SettingsManager.SETTING_VIDEOCAMERA_FLASH_MODE); - mKeyMap.put(SettingsManager.KEY_WHITE_BALANCE, SettingsManager.SETTING_WHITE_BALANCE); mKeyMap.put(SettingsManager.KEY_SCENE_MODE, SettingsManager.SETTING_SCENE_MODE); mKeyMap.put(SettingsManager.KEY_EXPOSURE, SettingsManager.SETTING_EXPOSURE); - mKeyMap.put(SettingsManager.KEY_TIMER, SettingsManager.SETTING_TIMER); - mKeyMap.put(SettingsManager.KEY_TIMER_SOUND_EFFECTS, - SettingsManager.SETTING_TIMER_SOUND_EFFECTS); mKeyMap.put(SettingsManager.KEY_VIDEO_EFFECT, SettingsManager.SETTING_VIDEO_EFFECT); mKeyMap.put(SettingsManager.KEY_CAMERA_ID, SettingsManager.SETTING_CAMERA_ID); mKeyMap.put(SettingsManager.KEY_CAMERA_HDR, SettingsManager.SETTING_CAMERA_HDR); @@ -185,16 +181,10 @@ public class SettingsCache { return SettingsManager.getFlashSetting(mContext); case SettingsManager.SETTING_VIDEOCAMERA_FLASH_MODE: return SettingsManager.getVideoFlashSetting(mContext); - case SettingsManager.SETTING_WHITE_BALANCE: - return SettingsManager.getWhiteBalanceSetting(mContext); case SettingsManager.SETTING_SCENE_MODE: return SettingsManager.getSceneModeSetting(mContext); case SettingsManager.SETTING_EXPOSURE: return SettingsManager.getExposureSetting(mContext, mCapabilities); - case SettingsManager.SETTING_TIMER: - return SettingsManager.getTimerSetting(mContext); - case SettingsManager.SETTING_TIMER_SOUND_EFFECTS: - return SettingsManager.getTimerSoundSetting(mContext); case SettingsManager.SETTING_VIDEO_EFFECT: return SettingsManager.getVideoEffectSetting(mContext); case SettingsManager.SETTING_CAMERA_ID: diff --git a/src/com/android/camera/settings/SettingsManager.java b/src/com/android/camera/settings/SettingsManager.java index 2fd9d541e..187bfd83b 100644 --- a/src/com/android/camera/settings/SettingsManager.java +++ b/src/com/android/camera/settings/SettingsManager.java @@ -350,11 +350,8 @@ public class SettingsManager { public static final int SETTING_FOCUS_MODE = 7; public static final int SETTING_FLASH_MODE = 8; public static final int SETTING_VIDEOCAMERA_FLASH_MODE = 9; - public static final int SETTING_WHITE_BALANCE = 10; public static final int SETTING_SCENE_MODE = 11; public static final int SETTING_EXPOSURE = 12; - public static final int SETTING_TIMER = 13; - public static final int SETTING_TIMER_SOUND_EFFECTS = 14; public static final int SETTING_VIDEO_EFFECT = 15; public static final int SETTING_CAMERA_ID = 16; public static final int SETTING_CAMERA_HDR = 17; @@ -385,11 +382,8 @@ public class SettingsManager { public static final String KEY_FOCUS_MODE = "pref_camera_focusmode_key"; public static final String KEY_FLASH_MODE = "pref_camera_flashmode_key"; public static final String KEY_VIDEOCAMERA_FLASH_MODE = "pref_camera_video_flashmode_key"; - public static final String KEY_WHITE_BALANCE = "pref_camera_whitebalance_key"; public static final String KEY_SCENE_MODE = "pref_camera_scenemode_key"; public static final String KEY_EXPOSURE = "pref_camera_exposure_key"; - public static final String KEY_TIMER = "pref_camera_timer_key"; - public static final String KEY_TIMER_SOUND_EFFECTS = "pref_camera_timer_sound_key"; public static final String KEY_VIDEO_EFFECT = "pref_video_effect_key"; public static final String KEY_CAMERA_ID = "pref_camera_id_key"; public static final String KEY_CAMERA_HDR = "pref_camera_hdr_key"; @@ -957,14 +951,6 @@ public class SettingsManager { values, FLUSH_ON); } - public static Setting getWhiteBalanceSetting(Context context) { - String defaultValue = context.getString(R.string.pref_camera_whitebalance_default); - String[] values = context.getResources().getStringArray( - R.array.pref_camera_whitebalance_entryvalues); - return new Setting(SOURCE_CAMERA, TYPE_STRING, defaultValue, KEY_WHITE_BALANCE, - values, FLUSH_OFF); - } - public static Setting getHdrSetting(Context context) { String defaultValue = context.getString(R.string.pref_camera_hdr_default); String[] values = context.getResources().getStringArray( @@ -1023,21 +1009,6 @@ public class SettingsManager { values, FLUSH_OFF); } - public static Setting getTimerSetting(Context context) { - String defaultValue = context.getString(R.string.pref_camera_timer_default); - String[] values = null; // TODO: get the values dynamically. - return new Setting(SOURCE_GLOBAL, TYPE_STRING, defaultValue, KEY_TIMER, - values, FLUSH_OFF); - } - - public static Setting getTimerSoundSetting(Context context) { - String defaultValue = context.getString(R.string.pref_camera_timer_sound_default); - String[] values = context.getResources().getStringArray( - R.array.pref_camera_timer_sound_entryvalues); - return new Setting(SOURCE_GLOBAL, TYPE_STRING, defaultValue, KEY_TIMER_SOUND_EFFECTS, - values, FLUSH_OFF); - } - public static Setting getVideoQualityBackSetting(Context context) { String defaultValue = context.getString(R.string.pref_video_quality_default); String[] values = context.getResources().getStringArray( @@ -1163,6 +1134,14 @@ public class SettingsManager { * Returns whether hdr plus mode is set on. */ public boolean isHdrPlusOn() { + String hdrOn = get(SettingsManager.SETTING_CAMERA_HDR_PLUS); + return hdrOn.equals(SettingsManager.VALUE_ON); + } + + /** + * Returns whether hdr mode is set on. + */ + public boolean isHdrOn() { String hdrOn = get(SettingsManager.SETTING_CAMERA_HDR); return hdrOn.equals(SettingsManager.VALUE_ON); } diff --git a/src/com/android/camera/widget/IndicatorIconController.java b/src/com/android/camera/widget/IndicatorIconController.java index e226ccddb..e855acf99 100644 --- a/src/com/android/camera/widget/IndicatorIconController.java +++ b/src/com/android/camera/widget/IndicatorIconController.java @@ -198,7 +198,7 @@ public class IndicatorIconController if (buttonManager.isEnabled(ButtonManager.BUTTON_HDRPLUS) && buttonManager.isVisible(ButtonManager.BUTTON_HDRPLUS)) { setIndicatorState(mController.getSettingsManager(), - SettingsManager.SETTING_CAMERA_HDR, + SettingsManager.SETTING_CAMERA_HDR_PLUS, mHdrIndicator, mHdrPlusIndicatorIcons, false); } else if (buttonManager.isEnabled(ButtonManager.BUTTON_HDR) && buttonManager.isVisible(ButtonManager.BUTTON_HDR)) { @@ -274,6 +274,10 @@ public class IndicatorIconController syncFlashIndicator(); break; } + case SettingsManager.SETTING_CAMERA_HDR_PLUS: { + syncHdrIndicator(); + break; + } case SettingsManager.SETTING_CAMERA_HDR: { syncHdrIndicator(); break; |