summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Emilian Peev <epeev@google.com> 2024-08-23 18:43:19 +0000
committer Android (Google) Code Review <android-gerrit@google.com> 2024-08-23 18:43:19 +0000
commit44784a5a8b7761ccbcb64ab83041b02442d9ae2d (patch)
treec62f55b6dd1a00ffaaf23d9a08ab6aad905766c9
parent39513a31b51a7e65d0f53ca19021598ee229ef9e (diff)
Revert "Camera: Set device executor thread name"
This reverts commit 39513a31b51a7e65d0f53ca19021598ee229ef9e. Reason for revert: It appears this is causing memory usage regressions in Crystalball. More information in b/361348383 Change-Id: I90d0e67fe3c91a4d46c659d0484bfb206816a407
-rw-r--r--core/java/android/hardware/camera2/impl/CameraDeviceImpl.java10
1 files changed, 1 insertions, 9 deletions
diff --git a/core/java/android/hardware/camera2/impl/CameraDeviceImpl.java b/core/java/android/hardware/camera2/impl/CameraDeviceImpl.java
index d0b9c74bf7dc..1e7f70bf5a72 100644
--- a/core/java/android/hardware/camera2/impl/CameraDeviceImpl.java
+++ b/core/java/android/hardware/camera2/impl/CameraDeviceImpl.java
@@ -80,7 +80,6 @@ import java.util.Set;
import java.util.concurrent.Executor;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
-import java.util.concurrent.ThreadFactory;
import java.util.concurrent.atomic.AtomicBoolean;
/**
@@ -355,14 +354,7 @@ public class CameraDeviceImpl extends CameraDevice
mCameraId = cameraId;
if (Flags.singleThreadExecutor()) {
mDeviceCallback = new ClientStateCallback(executor, callback);
- mDeviceExecutor = Executors.newSingleThreadExecutor(new ThreadFactory() {
- @Override
- public Thread newThread(Runnable r) {
- Thread thread = Executors.defaultThreadFactory().newThread(r);
- thread.setName("CameraDeviceExecutor");
- return thread;
- }
- });
+ mDeviceExecutor = Executors.newSingleThreadExecutor();
} else {
mDeviceCallback = callback;
mDeviceExecutor = executor;