diff options
author | Angus Kong <shkong@google.com> | 2013-07-18 18:04:19 -0700 |
---|---|---|
committer | Angus Kong <shkong@google.com> | 2013-07-29 12:19:02 -0700 |
commit | 9ef9925131835743f5316393758c14532c06277d (patch) | |
tree | 1eec5764f7be86ae7083c9480093e407c582c5cf /src/com/android/camera/PhotoModule.java | |
parent | 9d1cd3ec768cac1142de259a676893a08141a6d3 (diff) | |
download | android_packages_apps_Snap-9ef9925131835743f5316393758c14532c06277d.tar.gz android_packages_apps_Snap-9ef9925131835743f5316393758c14532c06277d.tar.bz2 android_packages_apps_Snap-9ef9925131835743f5316393758c14532c06277d.zip |
Refactor CameraManager.
1. CameraManager should be the only class accessing android.hardware.Camera.
2. For potential future upgrade in Camera HAL and android.hardward.Camera API
upgrade, CameraManager should be just an interface instead of concrete
implementation.
3. waitDone() in CameraProxy is removed.
4. ShutterCallback, PreviewCallback, PictureCallback and AF Callbacks are
wrapped by our own interfaces.
Change-Id: I595da17a1a9c6d476ee805b71c7f45ebb609e465
Diffstat (limited to 'src/com/android/camera/PhotoModule.java')
-rw-r--r-- | src/com/android/camera/PhotoModule.java | 56 |
1 files changed, 29 insertions, 27 deletions
diff --git a/src/com/android/camera/PhotoModule.java b/src/com/android/camera/PhotoModule.java index 361f111ea..6140848ec 100644 --- a/src/com/android/camera/PhotoModule.java +++ b/src/com/android/camera/PhotoModule.java @@ -28,7 +28,6 @@ import android.graphics.Bitmap; import android.graphics.SurfaceTexture; import android.hardware.Camera.CameraInfo; import android.hardware.Camera.Parameters; -import android.hardware.Camera.PictureCallback; import android.hardware.Camera.Size; import android.hardware.Sensor; import android.hardware.SensorEvent; @@ -52,7 +51,11 @@ import android.view.SurfaceHolder; import android.view.View; import android.view.WindowManager; +import com.android.camera.CameraManager.CameraAFCallback; +import com.android.camera.CameraManager.CameraAFMoveCallback; +import com.android.camera.CameraManager.CameraPictureCallback; import com.android.camera.CameraManager.CameraProxy; +import com.android.camera.CameraManager.CameraShutterCallback; import com.android.camera.ui.CountDownView.OnCountDownFinishedListener; import com.android.camera.ui.PopupManager; import com.android.camera.ui.RotateTextToast; @@ -623,7 +626,7 @@ public class PhotoModule if (mCameraDevice == null || mCameraStartUpThread != null) return; - mCameraDevice.setPreviewDisplayAsync(holder); + mCameraDevice.setPreviewDisplay(holder); // This happens when onConfigurationChanged arrives, surface has been // destroyed, and there is no onFullScreenChanged. if (mCameraState == PREVIEW_STOPPED) { @@ -680,7 +683,7 @@ public class PhotoModule } private final class ShutterCallback - implements android.hardware.Camera.ShutterCallback { + implements CameraShutterCallback { private boolean mAnimateFlash; @@ -689,7 +692,7 @@ public class PhotoModule } @Override - public void onShutter() { + public void onShutter(CameraProxy camera) { mShutterCallbackTime = System.currentTimeMillis(); mShutterLag = mShutterCallbackTime - mCaptureStartTime; Log.v(TAG, "mShutterLag = " + mShutterLag + "ms"); @@ -699,10 +702,10 @@ public class PhotoModule } } - private final class PostViewPictureCallback implements PictureCallback { + private final class PostViewPictureCallback + implements CameraPictureCallback { @Override - public void onPictureTaken( - byte [] data, android.hardware.Camera camera) { + public void onPictureTaken(byte [] data, CameraProxy camera) { mPostViewPictureCallbackTime = System.currentTimeMillis(); Log.v(TAG, "mShutterToPostViewCallbackTime = " + (mPostViewPictureCallbackTime - mShutterCallbackTime) @@ -710,17 +713,18 @@ public class PhotoModule } } - private final class RawPictureCallback implements PictureCallback { + private final class RawPictureCallback + implements CameraPictureCallback { @Override - public void onPictureTaken( - byte [] rawData, android.hardware.Camera camera) { + public void onPictureTaken(byte [] rawData, CameraProxy camera) { mRawPictureCallbackTime = System.currentTimeMillis(); Log.v(TAG, "mShutterToRawCallbackTime = " + (mRawPictureCallbackTime - mShutterCallbackTime) + "ms"); } } - private final class JpegPictureCallback implements PictureCallback { + private final class JpegPictureCallback + implements CameraPictureCallback { Location mLocation; public JpegPictureCallback(Location loc) { @@ -728,8 +732,7 @@ public class PhotoModule } @Override - public void onPictureTaken( - final byte [] jpegData, final android.hardware.Camera camera) { + public void onPictureTaken(final byte [] jpegData, CameraProxy camera) { if (mPaused) { return; } @@ -841,11 +844,10 @@ public class PhotoModule } } - private final class AutoFocusCallback - implements android.hardware.Camera.AutoFocusCallback { + private final class AutoFocusCallback implements CameraAFCallback { @Override public void onAutoFocus( - boolean focused, android.hardware.Camera camera) { + boolean focused, CameraProxy camera) { if (mPaused) return; mAutoFocusTime = System.currentTimeMillis() - mFocusStartTime; @@ -857,10 +859,10 @@ public class PhotoModule @TargetApi(ApiHelper.VERSION_CODES.JELLY_BEAN) private final class AutoFocusMoveCallback - implements android.hardware.Camera.AutoFocusMoveCallback { + implements CameraAFMoveCallback { @Override public void onAutoFocusMoving( - boolean moving, android.hardware.Camera camera) { + boolean moving, CameraProxy camera) { mFocusManager.onAutoFocusMoving(moving); } } @@ -965,10 +967,10 @@ public class PhotoModule Util.setGpsParameters(mParameters, loc); mCameraDevice.setParameters(mParameters); - mCameraDevice.takePicture2(new ShutterCallback(!animateBefore), + mCameraDevice.takePicture(mHandler, + new ShutterCallback(!animateBefore), mRawPictureCallback, mPostViewPictureCallback, - new JpegPictureCallback(loc), mCameraState, - mFocusManager.getFocusState()); + new JpegPictureCallback(loc)); mNamedImages.nameNewImage(mContentResolver, mCaptureStartTime); @@ -1408,7 +1410,7 @@ public class PhotoModule @Override public void autoFocus() { mFocusStartTime = System.currentTimeMillis(); - mCameraDevice.autoFocus(mAutoFocusCallback); + mCameraDevice.autoFocus(mHandler, mAutoFocusCallback); setCameraState(FOCUSING); } @@ -1558,11 +1560,11 @@ public class PhotoModule mUI.setPreviewSize(mParameters.getPreviewSize()); Object st = mUI.getSurfaceTexture(); if (st != null) { - mCameraDevice.setPreviewTextureAsync((SurfaceTexture) st); + mCameraDevice.setPreviewTexture((SurfaceTexture) st); } Log.v(TAG, "startPreview"); - mCameraDevice.startPreviewAsync(); + mCameraDevice.startPreview(); mFocusManager.onPreviewStarted(); if (mSnapshotOnIdle) { @@ -1769,10 +1771,10 @@ public class PhotoModule @TargetApi(ApiHelper.VERSION_CODES.JELLY_BEAN) private void updateAutoFocusMoveCallback() { if (mParameters.getFocusMode().equals(Util.FOCUS_MODE_CONTINUOUS_PICTURE)) { - mCameraDevice.setAutoFocusMoveCallback( - (AutoFocusMoveCallback) mAutoFocusMoveCallback); + mCameraDevice.setAutoFocusMoveCallback(mHandler, + (CameraManager.CameraAFMoveCallback) mAutoFocusMoveCallback); } else { - mCameraDevice.setAutoFocusMoveCallback(null); + mCameraDevice.setAutoFocusMoveCallback(null, null); } } |