From ac1179e2c3356bb8eee2f91d62f8dd09f3801d20 Mon Sep 17 00:00:00 2001 From: Sultanxda Date: Tue, 8 Dec 2015 18:51:56 -0800 Subject: Snap: Separate default focus time between camera/video For the camera, the user expects a default tap-to-focus duration of 3 seconds with exposure locking. For the camcorder, however, the user expects a tap-to-focus duration of 0 seconds with no exposure locking, as the rapid change in exposure while recording would negatively impact the video. Separate the tap-to-focus option to reflect the recommended defaults. [mikeioannina]: Adapted to Snap Change-Id: I75bc0610f5d301c9728b4bc104b86e0fab5493fb Signed-off-by: Sultanxda --- res/values/cm_strings.xml | 1 + res/xml/video_preferences.xml | 2 +- src/com/android/camera/VideoModule.java | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/res/values/cm_strings.xml b/res/values/cm_strings.xml index 5b8ddf306..1f7992239 100644 --- a/res/values/cm_strings.xml +++ b/res/values/cm_strings.xml @@ -111,6 +111,7 @@ Touch focus duration 3000 + 200 0s 3s diff --git a/res/xml/video_preferences.xml b/res/xml/video_preferences.xml index 134ecabf1..d42ad9d59 100755 --- a/res/xml/video_preferences.xml +++ b/res/xml/video_preferences.xml @@ -191,7 +191,7 @@ camera:entryValues="@array/pref_video_focusmode_entryvalues" /> diff --git a/src/com/android/camera/VideoModule.java b/src/com/android/camera/VideoModule.java index d91d361af..02246d229 100644 --- a/src/com/android/camera/VideoModule.java +++ b/src/com/android/camera/VideoModule.java @@ -2841,7 +2841,7 @@ public class VideoModule implements CameraModule, // Set focus time. mFocusManager.setFocusTime(Integer.decode( mPreferences.getString(CameraSettings.KEY_VIDEOCAMERA_FOCUS_TIME, - mActivity.getString(R.string.pref_camera_focustime_default)))); + mActivity.getString(R.string.pref_camera_video_focustime_default)))); // Set face detetction parameter. String faceDetection = mPreferences.getString( -- cgit v1.2.3