am fd6f316a: Merge "Ensure CaptureModule/API2 works without front camera" into ub...

* commit 'fd6f316adc5abfa071534642688bdf46f47a3925':
  Ensure CaptureModule/API2 works without front camera

diff --git a/src/com/android/camera/CaptureModule.java b/src/com/android/camera/CaptureModule.java
index a3f728c..640da4f 100644 (file)
--- a/src/com/android/camera/CaptureModule.java
+++ b/src/com/android/camera/CaptureModule.java
@@ -773,7 +773,7 @@ public class CaptureModule extends CameraModule implements
         return new HardwareSpec() {
             @Override
             public boolean isFrontCameraSupported() {
-                return true;
+                return mOneCameraManager.hasCameraFacing(Facing.FRONT);
             }
 
             @Override

diff --git a/src/com/android/camera/app/FirstRunDialog.java b/src/com/android/camera/app/FirstRunDialog.java
index c3a3f88..d810c4b 100644 (file)
--- a/src/com/android/camera/app/FirstRunDialog.java
+++ b/src/com/android/camera/app/FirstRunDialog.java
@@ -21,6 +21,7 @@ import android.content.Context;
 import android.content.DialogInterface;
 import android.view.ViewGroup;
 
+import com.android.camera.device.CameraId;
 import com.android.camera.exif.Rational;
 import com.android.camera.one.OneCamera.Facing;
 import com.android.camera.one.OneCameraAccessException;
@@ -157,10 +158,14 @@ public class FirstRunDialog {
             public void onConfirm(Rational aspectRatio) {
                 // Change resolution setting based on the chosen aspect ratio.
                 try {
-                    mResolutionSetting.setPictureAspectRatio(
-                          mOneCameraManager.findFirstCameraFacing(Facing.BACK), aspectRatio);
-                    mResolutionSetting.setPictureAspectRatio(
-                          mOneCameraManager.findFirstCameraFacing(Facing.FRONT), aspectRatio);
+                    CameraId backCameraId = mOneCameraManager.findFirstCameraFacing(Facing.BACK);
+                    if (backCameraId != null) {
+                        mResolutionSetting.setPictureAspectRatio(backCameraId, aspectRatio);
+                    }
+                    CameraId frontCameraId = mOneCameraManager.findFirstCameraFacing(Facing.FRONT);
+                    if (frontCameraId != null) {
+                        mResolutionSetting.setPictureAspectRatio(frontCameraId, aspectRatio);
+                    }
                 } catch (OneCameraAccessException ex) {
                     mListener.onCameraAccessException();
                     return;

diff --git a/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java b/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java
index 0b766a5..a4b5a9b 100644 (file)
--- a/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java
+++ b/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java
@@ -99,7 +99,8 @@ public class Camera2OneCameraManagerImpl implements OneCameraManager {
 
     @Override
     public CameraId findFirstCameraFacing(@Nonnull Facing facing) {
-        return CameraId.from(findCameraId(facing));
+        String cameraId = findCameraId(facing);
+        return (cameraId != null) ? CameraId.from(cameraId) : null;
     }
 
     @Override