diff options
author | Angus Kong <shkong@google.com> | 2014-06-16 16:07:28 -0700 |
---|---|---|
committer | Angus Kong <shkong@google.com> | 2014-06-19 10:35:12 -0700 |
commit | 831347d9cb5c0e3f03db7aadfc89e91b231104e7 (patch) | |
tree | a6ee25b99851a96ee1fa86be60c28127bd85461c /src/com/android/camera/hardware/HardwareSpecImpl.java | |
parent | ca8d97fda9470f575de1f68ee90d6ab0047644e0 (diff) | |
download | android_packages_apps_Camera2-831347d9cb5c0e3f03db7aadfc89e91b231104e7.tar.gz android_packages_apps_Camera2-831347d9cb5c0e3f03db7aadfc89e91b231104e7.tar.bz2 android_packages_apps_Camera2-831347d9cb5c0e3f03db7aadfc89e91b231104e7.zip |
Migrate to use new CameraSettings.
Totally get rid of Camera.Parameters.
Change-Id: I147b88f77ecab52600088a10417c3eac3d7fd340
Diffstat (limited to 'src/com/android/camera/hardware/HardwareSpecImpl.java')
-rw-r--r-- | src/com/android/camera/hardware/HardwareSpecImpl.java | 40 |
1 files changed, 7 insertions, 33 deletions
diff --git a/src/com/android/camera/hardware/HardwareSpecImpl.java b/src/com/android/camera/hardware/HardwareSpecImpl.java index 0ea5375a6..1e67dd686 100644 --- a/src/com/android/camera/hardware/HardwareSpecImpl.java +++ b/src/com/android/camera/hardware/HardwareSpecImpl.java @@ -16,14 +16,10 @@ package com.android.camera.hardware; -import android.hardware.Camera; - -import com.android.camera.util.CameraUtil; +import com.android.camera.app.CameraProvider; import com.android.camera.util.GcamHelper; import com.android.ex.camera2.portability.CameraCapabilities; -import java.util.List; - /** * HardwareSpecImpl is the default implementation of * {@link com.android.camera.hardware.HardwareSpec} for @@ -40,31 +36,10 @@ public class HardwareSpecImpl implements HardwareSpec { /** * Compute the supported values for all * {@link com.android.camera.hardware.HardwareSpec} methods - * based on {@link android.hardware.Camera.Parameters}. - */ - @Deprecated - public HardwareSpecImpl(Camera.Parameters parameters) { - // Cache whether front camera is supported. - mIsFrontCameraSupported = (Camera.getNumberOfCameras() > 1); - - // Cache whether hdr is supported. - mIsHdrSupported = CameraUtil.isCameraHdrSupported(parameters); - - // Cache whether hdr plus is supported. - mIsHdrPlusSupported = GcamHelper.hasGcamCapture(); - - // Cache whether flash is supported. - mIsFlashSupported = isFlashSupported(parameters); - } - - /** - * Compute the supported values for all - * {@link com.android.camera.hardware.HardwareSpec} methods - * based on {@link com.android.ex.camera2.portability.CameraCapabilities}. */ - public HardwareSpecImpl(CameraCapabilities capabilities) { + public HardwareSpecImpl(CameraProvider provider, CameraCapabilities capabilities) { // Cache whether front camera is supported. - mIsFrontCameraSupported = (Camera.getNumberOfCameras() > 1); + mIsFrontCameraSupported = (provider.getFirstFrontCameraId() != -1); // Cache whether hdr is supported. mIsHdrSupported = capabilities.supports(CameraCapabilities.SceneMode.HDR); @@ -73,8 +48,7 @@ public class HardwareSpecImpl implements HardwareSpec { mIsHdrPlusSupported = GcamHelper.hasGcamCapture(); // Cache whether flash is supported. - mIsFlashSupported = capabilities.supports(CameraCapabilities.FlashMode.AUTO) || - capabilities.supports(CameraCapabilities.FlashMode.ON); + mIsFlashSupported = isFlashSupported(capabilities); } @Override @@ -101,8 +75,8 @@ public class HardwareSpecImpl implements HardwareSpec { * Returns whether flash is supported and flash has more than * one possible value. */ - private boolean isFlashSupported(Camera.Parameters parameters) { - List<String> supportedFlashModes = parameters.getSupportedFlashModes(); - return !(supportedFlashModes == null || (supportedFlashModes.size() == 1)); + private boolean isFlashSupported(CameraCapabilities capabilities) { + return (capabilities.supports(CameraCapabilities.FlashMode.AUTO) || capabilities.supports + (CameraCapabilities.FlashMode.ON)); } } |