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

Commit 500052d8 authored by Ram Mohan M's avatar Ram Mohan M Committed by Pawin Vongmasa
Browse files

Extend adaptive playback test for graphic buffer modes 

Test: make vts -j99 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 \
&& vts-tradefed run commandAndExit vts \
--skip-all-system-status-check --primary-abi-only \
--skip-preconditions --module VtsHalMediaOmxV1_0Host \
-l INFO

Bug: 63796949

Change-Id: I4a4aad3a0dd4f528c8e855164cbb2998b33be1cd
parent 95736780

media/omx/1.0/vts/functional/video/VtsHalMediaOmxV1_0TargetVideoDecTest.cpp

+11 −2
Original line number Diff line number Diff line
@@ -946,10 +946,16 @@ TEST_F(VideoDecHidlTest, AdaptivePlaybackTest) { 
    }



    // set port mode

    portMode[0] = PortMode::PRESET_BYTE_BUFFER;

    portMode[1] = PortMode::DYNAMIC_ANW_BUFFER;

    status = omxNode->setPortMode(kPortIndexInput, portMode[0]);

    ASSERT_EQ(status, ::android::hardware::media::omx::V1_0::Status::OK);

    status = omxNode->setPortMode(kPortIndexOutput, portMode[1]);

    if (status != ::android::hardware::media::omx::V1_0::Status::OK) {

        portMode[1] = PortMode::PRESET_BYTE_BUFFER;

        status = omxNode->setPortMode(kPortIndexOutput, portMode[1]);

        ASSERT_EQ(status, ::android::hardware::media::omx::V1_0::Status::OK);

    }



    // prepare for adaptive playback

    uint32_t adaptiveMaxWidth = 320;
@@ -961,7 +967,10 @@ TEST_F(VideoDecHidlTest, AdaptivePlaybackTest) { 
        // support for adaptive play back is mandatory in Byte Buffer mode

        ASSERT_EQ(status, ::android::hardware::media::omx::V1_0::Status::OK);

    } else {

        return;

        // for vendor codecs, support for adaptive play back is optional

        // in byte buffer mode.

        if (portMode[1] == PortMode::PRESET_BYTE_BUFFER) return;

        if (status != ::android::hardware::media::omx::V1_0::Status::OK) return;

    }



    // TODO: Handle this better !!!