Revert "Convert from HIDL mapper to libui GraphicBufferMapper"
Revert submission 24747509-camera_fail2
Reason for revert: DroidMonitor-triggered revert due to breakage
b/302990858.
This may cause CtsAppOpsTestCases b/300115646 to fail again,
unfortunately (b/300115646).
Bug: 302990858
Reverted changes: /q/submissionid:24747509-camera_fail2
Change-Id: I0191b3ee88846f5b2c6e37ebd6be6d1369d9dba9
diff --git a/camera/device/1.0/default/Android.bp b/camera/device/1.0/default/Android.bp
index 6992ff0..9ff6480 100644
--- a/camera/device/1.0/default/Android.bp
+++ b/camera/device/1.0/default/Android.bp
@@ -32,7 +32,6 @@
"libgralloctypes",
"libhardware",
"libcamera_metadata",
- "libui",
],
static_libs: [
"android.hardware.camera.common@1.0-helper",
diff --git a/camera/device/3.2/default/Android.bp b/camera/device/3.2/default/Android.bp
index adf834a..a196291 100644
--- a/camera/device/3.2/default/Android.bp
+++ b/camera/device/3.2/default/Android.bp
@@ -30,7 +30,6 @@
"libhardware",
"libcamera_metadata",
"libfmq",
- "libui",
],
static_libs: [
"android.hardware.camera.common@1.0-helper",
diff --git a/camera/device/3.4/default/Android.bp b/camera/device/3.4/default/Android.bp
index 100106e..9f0c777 100644
--- a/camera/device/3.4/default/Android.bp
+++ b/camera/device/3.4/default/Android.bp
@@ -106,7 +106,6 @@
"libjpeg",
"libexif",
"libtinyxml2",
- "libui",
],
static_libs: [
"android.hardware.camera.common@1.0-helper",
diff --git a/camera/device/3.4/default/ExternalCameraDeviceSession.cpp b/camera/device/3.4/default/ExternalCameraDeviceSession.cpp
index 01b3d41..ca7186b 100644
--- a/camera/device/3.4/default/ExternalCameraDeviceSession.cpp
+++ b/camera/device/3.4/default/ExternalCameraDeviceSession.cpp
@@ -1574,23 +1574,14 @@
} break;
case PixelFormat::YCBCR_420_888:
case PixelFormat::YV12: {
- android::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
- static_cast<int32_t>(halBuf.height)};
- android_ycbcr result =
- sHandleImporter.lockYCbCr(*(halBuf.bufPtr), halBuf.usage, outRect);
- ALOGV("%s: outLayout y %p cb %p cr %p y_str %zu c_str %zu c_step %zu", __FUNCTION__,
- result.y, result.cb, result.cr, result.ystride, result.cstride,
- result.chroma_step);
- if (result.ystride > UINT32_MAX || result.cstride > UINT32_MAX ||
- result.chroma_step > UINT32_MAX) {
- return onDeviceError("%s: lockYCbCr failed. Unexpected values!", __FUNCTION__);
- }
- YCbCrLayout outLayout = {.y = result.y,
- .cb = result.cb,
- .cr = result.cr,
- .yStride = static_cast<uint32_t>(result.ystride),
- .cStride = static_cast<uint32_t>(result.cstride),
- .chromaStep = static_cast<uint32_t>(result.chroma_step)};
+ IMapper::Rect outRect {0, 0,
+ static_cast<int32_t>(halBuf.width),
+ static_cast<int32_t>(halBuf.height)};
+ YCbCrLayout outLayout = sHandleImporter.lockYCbCr(
+ *(halBuf.bufPtr), halBuf.usage, outRect);
+ ALOGV("%s: outLayout y %p cb %p cr %p y_str %d c_str %d c_step %d",
+ __FUNCTION__, outLayout.y, outLayout.cb, outLayout.cr,
+ outLayout.yStride, outLayout.cStride, outLayout.chromaStep);
// Convert to output buffer size/format
uint32_t outputFourcc = getFourCcFromLayout(outLayout);
diff --git a/camera/device/3.5/default/Android.bp b/camera/device/3.5/default/Android.bp
index bc15629..9d27b32 100644
--- a/camera/device/3.5/default/Android.bp
+++ b/camera/device/3.5/default/Android.bp
@@ -46,7 +46,6 @@
],
shared_libs: [
"libhidlbase",
- "libui",
"libutils",
"libcutils",
"camera.device@3.2-impl",
@@ -82,7 +81,6 @@
],
shared_libs: [
"libhidlbase",
- "libui",
"libutils",
"libcutils",
"camera.device@3.2-impl",
diff --git a/camera/device/3.6/default/Android.bp b/camera/device/3.6/default/Android.bp
index b4a486f..89ee145 100644
--- a/camera/device/3.6/default/Android.bp
+++ b/camera/device/3.6/default/Android.bp
@@ -41,7 +41,6 @@
],
shared_libs: [
"libhidlbase",
- "libui",
"libutils",
"libcutils",
"camera.device@3.2-impl",
diff --git a/camera/device/3.6/default/ExternalCameraOfflineSession.cpp b/camera/device/3.6/default/ExternalCameraOfflineSession.cpp
index 1f1dfee..e606fda 100644
--- a/camera/device/3.6/default/ExternalCameraOfflineSession.cpp
+++ b/camera/device/3.6/default/ExternalCameraOfflineSession.cpp
@@ -222,23 +222,14 @@
} break;
case PixelFormat::YCBCR_420_888:
case PixelFormat::YV12: {
- android::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
- static_cast<int32_t>(halBuf.height)};
- android_ycbcr result =
- sHandleImporter.lockYCbCr(*(halBuf.bufPtr), halBuf.usage, outRect);
- ALOGV("%s: outLayout y %p cb %p cr %p y_str %zu c_str %zu c_step %zu", __FUNCTION__,
- result.y, result.cb, result.cr, result.ystride, result.cstride,
- result.chroma_step);
- if (result.ystride > UINT32_MAX || result.cstride > UINT32_MAX ||
- result.chroma_step > UINT32_MAX) {
- return onDeviceError("%s: lockYCbCr failed. Unexpected values!", __FUNCTION__);
- }
- YCbCrLayout outLayout = {.y = result.y,
- .cb = result.cb,
- .cr = result.cr,
- .yStride = static_cast<uint32_t>(result.ystride),
- .cStride = static_cast<uint32_t>(result.cstride),
- .chromaStep = static_cast<uint32_t>(result.chroma_step)};
+ IMapper::Rect outRect {0, 0,
+ static_cast<int32_t>(halBuf.width),
+ static_cast<int32_t>(halBuf.height)};
+ YCbCrLayout outLayout = sHandleImporter.lockYCbCr(
+ *(halBuf.bufPtr), halBuf.usage, outRect);
+ ALOGV("%s: outLayout y %p cb %p cr %p y_str %d c_str %d c_step %d",
+ __FUNCTION__, outLayout.y, outLayout.cb, outLayout.cr,
+ outLayout.yStride, outLayout.cStride, outLayout.chromaStep);
// Convert to output buffer size/format
uint32_t outputFourcc = V3_4::implementation::getFourCcFromLayout(outLayout);
diff --git a/camera/device/default/Android.bp b/camera/device/default/Android.bp
index 5fbcb5d..b9f10d6 100644
--- a/camera/device/default/Android.bp
+++ b/camera/device/default/Android.bp
@@ -40,7 +40,7 @@
shared_libs: [
"android.hardware.camera.common-V1-ndk",
"android.hardware.camera.device-V1-ndk",
- "android.hardware.graphics.allocator-V2-ndk",
+ "android.hardware.graphics.allocator-V1-ndk",
"android.hardware.graphics.mapper@2.0",
"android.hardware.graphics.mapper@3.0",
"android.hardware.graphics.mapper@4.0",
@@ -60,7 +60,6 @@
"libsync",
"libtinyxml2",
"libutils",
- "libui",
"libyuv",
],
static_libs: [
diff --git a/camera/device/default/ExternalCameraDeviceSession.cpp b/camera/device/default/ExternalCameraDeviceSession.cpp
index 896e0da..95a36f0 100644
--- a/camera/device/default/ExternalCameraDeviceSession.cpp
+++ b/camera/device/default/ExternalCameraDeviceSession.cpp
@@ -2882,23 +2882,13 @@
} break;
case PixelFormat::YCBCR_420_888:
case PixelFormat::YV12: {
- android::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
+ IMapper::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
static_cast<int32_t>(halBuf.height)};
- android_ycbcr result = sHandleImporter.lockYCbCr(
+ YCbCrLayout outLayout = sHandleImporter.lockYCbCr(
*(halBuf.bufPtr), static_cast<uint64_t>(halBuf.usage), outRect);
- ALOGV("%s: outLayout y %p cb %p cr %p y_str %zu c_str %zu c_step %zu", __FUNCTION__,
- result.y, result.cb, result.cr, result.ystride, result.cstride,
- result.chroma_step);
- if (result.ystride > UINT32_MAX || result.cstride > UINT32_MAX ||
- result.chroma_step > UINT32_MAX) {
- return onDeviceError("%s: lockYCbCr failed. Unexpected values!", __FUNCTION__);
- }
- YCbCrLayout outLayout = {.y = result.y,
- .cb = result.cb,
- .cr = result.cr,
- .yStride = static_cast<uint32_t>(result.ystride),
- .cStride = static_cast<uint32_t>(result.cstride),
- .chromaStep = static_cast<uint32_t>(result.chroma_step)};
+ ALOGV("%s: outLayout y %p cb %p cr %p y_str %d c_str %d c_step %d", __FUNCTION__,
+ outLayout.y, outLayout.cb, outLayout.cr, outLayout.yStride, outLayout.cStride,
+ outLayout.chromaStep);
// Convert to output buffer size/format
uint32_t outputFourcc = getFourCcFromLayout(outLayout);
diff --git a/camera/device/default/ExternalCameraDeviceSession.h b/camera/device/default/ExternalCameraDeviceSession.h
index 736bfd1..836266f 100644
--- a/camera/device/default/ExternalCameraDeviceSession.h
+++ b/camera/device/default/ExternalCameraDeviceSession.h
@@ -24,9 +24,6 @@
#include <aidl/android/hardware/camera/device/BufferRequest.h>
#include <aidl/android/hardware/camera/device/Stream.h>
#include <android-base/unique_fd.h>
-#include <android/hardware/graphics/mapper/2.0/IMapper.h>
-#include <android/hardware/graphics/mapper/3.0/IMapper.h>
-#include <android/hardware/graphics/mapper/4.0/IMapper.h>
#include <fmq/AidlMessageQueue.h>
#include <utils/Thread.h>
#include <deque>
@@ -58,7 +55,6 @@
using ::android::hardware::camera::common::helper::SimpleThread;
using ::android::hardware::camera::external::common::ExternalCameraConfig;
using ::android::hardware::camera::external::common::SizeHasher;
-using ::android::hardware::graphics::mapper::V2_0::YCbCrLayout;
using ::ndk::ScopedAStatus;
class ExternalCameraDeviceSession : public BnCameraDeviceSession, public OutputThreadInterface {
diff --git a/camera/device/default/ExternalCameraOfflineSession.cpp b/camera/device/default/ExternalCameraOfflineSession.cpp
index 53bd44f..4c7f732 100644
--- a/camera/device/default/ExternalCameraOfflineSession.cpp
+++ b/camera/device/default/ExternalCameraOfflineSession.cpp
@@ -486,23 +486,13 @@
} break;
case PixelFormat::YCBCR_420_888:
case PixelFormat::YV12: {
- android::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
+ IMapper::Rect outRect{0, 0, static_cast<int32_t>(halBuf.width),
static_cast<int32_t>(halBuf.height)};
- android_ycbcr result = sHandleImporter.lockYCbCr(
+ YCbCrLayout outLayout = sHandleImporter.lockYCbCr(
*(halBuf.bufPtr), static_cast<uint64_t>(halBuf.usage), outRect);
- ALOGV("%s: outLayout y %p cb %p cr %p y_str %zu c_str %zu c_step %zu", __FUNCTION__,
- result.y, result.cb, result.cr, result.ystride, result.cstride,
- result.chroma_step);
- if (result.ystride > UINT32_MAX || result.cstride > UINT32_MAX ||
- result.chroma_step > UINT32_MAX) {
- return onDeviceError("%s: lockYCbCr failed. Unexpected values!", __FUNCTION__);
- }
- YCbCrLayout outLayout = {.y = result.y,
- .cb = result.cb,
- .cr = result.cr,
- .yStride = static_cast<uint32_t>(result.ystride),
- .cStride = static_cast<uint32_t>(result.cstride),
- .chromaStep = static_cast<uint32_t>(result.chroma_step)};
+ ALOGV("%s: outLayout y %p cb %p cr %p y_str %d c_str %d c_step %d", __FUNCTION__,
+ outLayout.y, outLayout.cb, outLayout.cr, outLayout.yStride, outLayout.cStride,
+ outLayout.chromaStep);
// Convert to output buffer size/format
uint32_t outputFourcc = getFourCcFromLayout(outLayout);
@@ -554,4 +544,4 @@
} // namespace device
} // namespace camera
} // namespace hardware
-} // namespace android
+} // namespace android
\ No newline at end of file
diff --git a/camera/device/default/ExternalCameraUtils.h b/camera/device/default/ExternalCameraUtils.h
index d434905..b37933c 100644
--- a/camera/device/default/ExternalCameraUtils.h
+++ b/camera/device/default/ExternalCameraUtils.h
@@ -25,11 +25,7 @@
#include <aidl/android/hardware/camera/device/NotifyMsg.h>
#include <aidl/android/hardware/graphics/common/BufferUsage.h>
#include <aidl/android/hardware/graphics/common/PixelFormat.h>
-#include <android/hardware/graphics/mapper/2.0/IMapper.h>
-#include <android/hardware/graphics/mapper/3.0/IMapper.h>
-#include <android/hardware/graphics/mapper/4.0/IMapper.h>
#include <tinyxml2.h>
-#include <map>
#include <unordered_map>
#include <unordered_set>
@@ -41,8 +37,6 @@
using ::aidl::android::hardware::graphics::common::PixelFormat;
using ::android::hardware::camera::common::V1_0::helper::CameraMetadata;
using ::android::hardware::camera::common::V1_0::helper::HandleImporter;
-using ::android::hardware::graphics::mapper::V2_0::IMapper;
-using ::android::hardware::graphics::mapper::V2_0::YCbCrLayout;
namespace android {
namespace hardware {