diff options
author | 2018-11-16 16:28:08 -0800 | |
---|---|---|
committer | 2018-11-28 15:57:21 -0800 | |
commit | 069e8389df7aee7e8812a036bcda43c3fe24cabc (patch) | |
tree | 5c50727256298ce4b603af94e7f2be5df0d7d3f4 /libs/ui/BufferHubBuffer.cpp | |
parent | 910c0aa1384ee3122a4d3929ca3b94943a3dc539 (diff) |
Move BufferHubMetadata off pdx::fileHandle
Use android::base::unique_fd to replace LocalHandle. For BorrowedHandle,
a const reference of the unique_fd is returned.
Updated the test cases to fit in current behavior.
Test: BufferHubMetadata_test, BufferHubBuffer_test, BufferNode_test,
buffer_hub-test (passed)
Fix: 118888072
Change-Id: I34de335ed9a10864ac226cd4d7d261ba0078045d
Diffstat (limited to 'libs/ui/BufferHubBuffer.cpp')
-rw-r--r-- | libs/ui/BufferHubBuffer.cpp | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/libs/ui/BufferHubBuffer.cpp b/libs/ui/BufferHubBuffer.cpp index 8cc1a4e3b4..dd79775d01 100644 --- a/libs/ui/BufferHubBuffer.cpp +++ b/libs/ui/BufferHubBuffer.cpp @@ -36,10 +36,12 @@ #include <private/dvr/bufferhub_rpc.h> #pragma clang diagnostic pop -#include <ui/BufferHubBuffer.h> - #include <poll.h> +#include <android-base/unique_fd.h> +#include <ui/BufferHubBuffer.h> + +using android::base::unique_fd; using android::dvr::BufferTraits; using android::dvr::DetachedBufferRPC; using android::dvr::NativeHandleWrapper; @@ -132,7 +134,9 @@ int BufferHubBuffer::ImportGraphicBuffer() { const int bufferId = bufferTraits.id(); // Import the metadata. - mMetadata = BufferHubMetadata::Import(bufferTraits.take_metadata_handle()); + LocalHandle metadataHandle = bufferTraits.take_metadata_handle(); + unique_fd metadataFd(metadataHandle.Release()); + mMetadata = BufferHubMetadata::Import(std::move(metadataFd)); if (!mMetadata.IsValid()) { ALOGE("BufferHubBuffer::ImportGraphicBuffer: invalid metadata."); |