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

Commit 58965d35 authored by ApurupaPattapu's avatar ApurupaPattapu Committed by Steve Kondik
Browse files

frameworks/av: Set arbitrary mode to false by default 

When arbitrary mode key is not set by extractor, use frame
by frame as default mode

CRs-Fixed: 512518
Change-Id: Iae20e573d7bb763c44d9586bf1e6ef4e60f00e96
parent f79d40e3

media/libstagefright/ExtendedCodec.cpp

+7 −7
Original line number Diff line number Diff line
@@ -282,16 +282,14 @@ void ExtendedCodec::configureVideoCodec( 
        return;

    }



    int32_t arbitraryMode = 1;

    int32_t arbitraryMode = 0;

    bool success = meta->findInt32(kKeyUseArbitraryMode, &arbitraryMode);

    bool useArbitraryMode = true;

    if (success) {

        useArbitraryMode = arbitraryMode ? true : false;

    bool useFrameByFrameMode = true; //default option

    if (success && arbitraryMode) {

        useFrameByFrameMode = false;

    }



    if (useArbitraryMode) {

        ALOGI("Decoder should be in arbitrary mode");

    } else{

    if (useFrameByFrameMode) {

        ALOGI("Enable frame by frame mode");

        OMX_QCOM_PARAM_PORTDEFINITIONTYPE portFmt;

        portFmt.nPortIndex = kPortIndexInput;
@@ -301,6 +299,8 @@ void ExtendedCodec::configureVideoCodec( 
        if(err != OK) {

            ALOGW("Failed to set frame packing format on component");

        }

    } else {

        ALOGI("Decoder should be in arbitrary mode");

    }



    // Enable timestamp reordering only for AVI/mpeg4 and vc1 clips