commit e129404551e0065e4cdce48806c7be33f5af9433
author Wang Wenbin <wenbinw@codeaurora.org> Sat Sep 14 13:38:34 2013 +0800
committer Whitehawkx <xxwhitehawk@gmail.com> Tue Oct 01 07:09:28 2013 -0700
tree 494539a18e112e60eed31dc053a545eb590687c7
parent 501c443a49d121385ef71314a37e5ecbb3d4e0ec

camera: improve snapshot KPI

In camera snapshot, thunbnail slide animation is start in JPEG
callback, make it start in raw data callback so that it can make
use of JPEG encode time which in turn shorten the snapshot time.

Change-Id: I1ce8a768a1cb54e8244725bdcb1e49d3ca5a914b

src/com/android/camera/PhotoModule.java

diff --git a/src/com/android/camera/PhotoModule.java b/src/com/android/camera/PhotoModule.java
index d8dacb5..685bf95 100644
--- a/src/com/android/camera/PhotoModule.java
+++ b/src/com/android/camera/PhotoModule.java
@@ -783,6 +783,14 @@
             mRawPictureCallbackTime = System.currentTimeMillis();
             Log.v(TAG, "mShutterToRawCallbackTime = "
                     + (mRawPictureCallbackTime - mShutterCallbackTime) + "ms");
+            if (ApiHelper.HAS_SURFACE_TEXTURE && !mIsImageCaptureIntent
+                    && mActivity.mShowCameraAppView) {
+                // Finish capture animation
+                mHandler.removeMessages(CAPTURE_ANIMATION_DONE);
+                ((CameraScreenNail) mActivity.mCameraScreenNail).animateSlide();
+                mHandler.sendEmptyMessageDelayed(CAPTURE_ANIMATION_DONE,
+                        CaptureAnimManager.getAnimationDuration());
+            }
         }
     }
 
@@ -825,14 +833,6 @@
             // Only animate when in full screen capture mode
             // i.e. If monkey/a user swipes to the gallery during picture taking,
             // don't show animation
-            if (ApiHelper.HAS_SURFACE_TEXTURE && !mIsImageCaptureIntent
-                    && mActivity.mShowCameraAppView) {
-                // Finish capture animation
-                mHandler.removeMessages(CAPTURE_ANIMATION_DONE);
-                ((CameraScreenNail) mActivity.mCameraScreenNail).animateSlide();
-                mHandler.sendEmptyMessageDelayed(CAPTURE_ANIMATION_DONE,
-                        CaptureAnimManager.getAnimationDuration());
-            }
             mFocusManager.updateFocusUI(); // Ensure focus indicator is hidden.
 
             boolean isSamsungHDR =