summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTreehugger Robot <treehugger-gerrit@google.com>2020-12-17 02:44:02 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2020-12-17 02:44:02 +0000
commit2205669f74ec3e1bb8d913af0d1a690c4a4d3e3b (patch)
treeb2a577f4c1ef0d98c57d833fff41e9d3d72a7eda
parentbc84b4695cf503dc332c18f3ad1759320078530d (diff)
parent51988f34aada101cb91cb09ce4f57bad0f4e9904 (diff)
downloadCamera2-android10-gsi.tar.gz
Merge "Add external camera support" into android10-gsiandroid10-gsi
-rw-r--r--src/com/android/camera/DisableCameraReceiver.java7
-rw-r--r--src/com/android/camera/one/v2/Camera2OneCameraManagerImpl.java8
-rw-r--r--src/com/android/camera/settings/SettingsManager.java5
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);