Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6f9bf138 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "aidlpersistentsurface: enable submitting a null buffer" into main

parents 7ef477e1 d8160297

media/codec2/sfplugin/C2AidlNode.cpp

+7 −2
Original line number Original line Diff line number Diff line
@@ -68,10 +68,15 @@ C2AidlNode::C2AidlNode(const std::shared_ptr<Codec2Client::Component> &comp) 
}

}





::ndk::ScopedAStatus C2AidlNode::submitBuffer(

::ndk::ScopedAStatus C2AidlNode::submitBuffer(

        int32_t buffer, const ::aidl::android::hardware::HardwareBuffer& hBuffer,

        int32_t buffer,

        const std::optional<::aidl::android::hardware::HardwareBuffer>& hBuffer,

        int32_t flags, int64_t timestamp, const ::ndk::ScopedFileDescriptor& fence) {

        int32_t flags, int64_t timestamp, const ::ndk::ScopedFileDescriptor& fence) {

    sp<GraphicBuffer> gBuf;

    sp<GraphicBuffer> gBuf;

    AHardwareBuffer *ahwb = hBuffer.get();

    AHardwareBuffer *ahwb = nullptr;

    if (hBuffer.has_value()) {

        ahwb = hBuffer.value().get();

    }



    if (ahwb) {

    if (ahwb) {

        gBuf = AHardwareBuffer_to_GraphicBuffer(ahwb);

        gBuf = AHardwareBuffer_to_GraphicBuffer(ahwb);

    }

    }

media/codec2/sfplugin/C2AidlNode.h

+1 −1
Original line number Original line Diff line number Diff line
@@ -49,7 +49,7 @@ public: 




    ::ndk::ScopedAStatus submitBuffer(

    ::ndk::ScopedAStatus submitBuffer(

            int32_t buffer,

            int32_t buffer,

            const ::aidl::android::hardware::HardwareBuffer& hBuffer,

            const std::optional<::aidl::android::hardware::HardwareBuffer>& hBuffer,

            int32_t flags,

            int32_t flags,

            int64_t timestampUs,

            int64_t timestampUs,

            const ::ndk::ScopedFileDescriptor& fence) override;

            const ::ndk::ScopedFileDescriptor& fence) override;

media/module/aidlpersistentsurface/aidl/android/media/IAidlNode.aidl

+1 −1
Original line number Original line Diff line number Diff line
@@ -44,7 +44,7 @@ interface IAidlNode { 
    void setInputSurface(IAidlBufferSource bufferSource);

    void setInputSurface(IAidlBufferSource bufferSource);

    void submitBuffer(

    void submitBuffer(

            int buffer,

            int buffer,

            in HardwareBuffer hBuffer,

            in @nullable HardwareBuffer hBuffer,

            int flags,

            int flags,

            long timestampUs,

            long timestampUs,

            in @nullable ParcelFileDescriptor fence);

            in @nullable ParcelFileDescriptor fence);

media/module/aidlpersistentsurface/wrapper/WAidlGraphicBufferSource.cpp

+11 −6
Original line number Original line Diff line number Diff line
@@ -51,19 +51,24 @@ struct WAidlGraphicBufferSource::WAidlNodeWrapper : public IAidlNodeWrapper { 
            int32_t bufferId, uint32_t flags,

            int32_t bufferId, uint32_t flags,

            const sp<GraphicBuffer> &buffer,

            const sp<GraphicBuffer> &buffer,

            int64_t timestamp, int fenceFd) override {

            int64_t timestamp, int fenceFd) override {

        AHardwareBuffer *ahwBuffer = nullptr;

        ::ndk::ScopedFileDescriptor fence(fenceFd);

        ::aidl::android::hardware::HardwareBuffer hBuffer;

        if (buffer.get()) {

        if (buffer.get()) {

            ahwBuffer = AHardwareBuffer_from_GraphicBuffer(buffer.get());

            ::aidl::android::hardware::HardwareBuffer hBuffer;

            AHardwareBuffer *ahwBuffer = AHardwareBuffer_from_GraphicBuffer(buffer.get());

            AHardwareBuffer_acquire(ahwBuffer);

            AHardwareBuffer_acquire(ahwBuffer);

            hBuffer.reset(ahwBuffer);

            hBuffer.reset(ahwBuffer);

        }





        ::ndk::ScopedFileDescriptor fence(fenceFd);

            return fromAidlStatus(mNode->submitBuffer(

                    bufferId,

                    std::move(hBuffer),

                    flags,

                    timestamp,

                    fence));

        }





        return fromAidlStatus(mNode->submitBuffer(

        return fromAidlStatus(mNode->submitBuffer(

              bufferId,

              bufferId,

              hBuffer,

              {},

              flags,

              flags,

              timestamp,

              timestamp,

              fence));

              fence));