diff options
author | Treehugger Robot <treehugger-gerrit@google.com> | 2020-12-17 02:44:02 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2020-12-17 02:44:02 +0000 |
commit | 2205669f74ec3e1bb8d913af0d1a690c4a4d3e3b (patch) | |
tree | b2a577f4c1ef0d98c57d833fff41e9d3d72a7eda | |
parent | bc84b4695cf503dc332c18f3ad1759320078530d (diff) | |
parent | 51988f34aada101cb91cb09ce4f57bad0f4e9904 (diff) | |
download | Camera2-android10-gsi.tar.gz |
Merge "Add external camera support" into android10-gsiandroid10-gsi
-rw-r--r-- | src/com/android/camera/DisableCameraReceiver.java | 7 | ||||
-rw-r--r-- | src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java | 8 | ||||
-rw-r--r-- | src/com/android/camera/settings/SettingsManager.java | 5 |
3 files changed, 18 insertions, 2 deletions
diff --git a/src/com/android/camera/DisableCameraReceiver.java b/src/com/android/camera/DisableCameraReceiver.java index 5237c1d58..ed58783e9 100644 --- a/src/com/android/camera/DisableCameraReceiver.java +++ b/src/com/android/camera/DisableCameraReceiver.java @@ -42,7 +42,7 @@ public class DisableCameraReceiver extends BroadcastReceiver { ? hasBackCamera() : hasCamera(); - if (!needCameraActivity) { + if (!needCameraActivity && !supportExternalCamera(context)) { Log.i(TAG, "disable all camera activities"); for (int i = 0; i < ACTIVITIES.length; i++) { disableComponent(context, ACTIVITIES[i]); @@ -53,6 +53,11 @@ public class DisableCameraReceiver extends BroadcastReceiver { disableComponent(context, "com.android.camera.DisableCameraReceiver"); } + private boolean supportExternalCamera(Context context) { + PackageManager pm = context.getPackageManager(); + return pm.hasSystemFeature(PackageManager.FEATURE_CAMERA_EXTERNAL); + } + private boolean hasCamera() { int n = android.hardware.Camera.getNumberOfCameras(); Log.i(TAG, "number of camera: " + n); diff --git a/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java b/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java index 283ac677e..c7552450c 100644 --- a/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java +++ b/src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java @@ -157,6 +157,10 @@ public class Camera2OneCameraManagerImpl implements OneCameraManager { String cameraId = findFirstCameraIdFacing(CameraCharacteristics.LENS_FACING_BACK); if (cameraId == null) { Log.w(TAG, "No back-facing camera found."); + cameraId = findFirstCameraIdFacing(CameraCharacteristics.LENS_FACING_EXTERNAL); + if (cameraId == null) { + Log.w(TAG, "No external camera found."); + } } return cameraId; } @@ -167,6 +171,10 @@ public class Camera2OneCameraManagerImpl implements OneCameraManager { String cameraId = findFirstCameraIdFacing(CameraCharacteristics.LENS_FACING_FRONT); if (cameraId == null) { Log.w(TAG, "No front-facing camera found."); + cameraId = findFirstCameraIdFacing(CameraCharacteristics.LENS_FACING_EXTERNAL); + if (cameraId == null) { + Log.w(TAG, "No external camera found."); + } } return cameraId; } diff --git a/src/com/android/camera/settings/SettingsManager.java b/src/com/android/camera/settings/SettingsManager.java index c5c494ea8..d2794e578 100644 --- a/src/com/android/camera/settings/SettingsManager.java +++ b/src/com/android/camera/settings/SettingsManager.java @@ -128,8 +128,11 @@ public class SettingsManager { protected SharedPreferences openPreferences(String scope) { synchronized (mLock) { SharedPreferences preferences; + // For external camera, scope could have "/" separator which is a invalid path + // for the shared preference. + String validScope = scope.replaceAll("/", "_"); preferences = mContext.getSharedPreferences( - mPackageName + scope, Context.MODE_PRIVATE); + mPackageName + validScope, Context.MODE_PRIVATE); for (OnSharedPreferenceChangeListener listener : mSharedPreferenceListeners) { preferences.registerOnSharedPreferenceChangeListener(listener); |