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

Commit 4709170d authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Codec2Buffer: accept RGB/RGBA MediaImage layouts" am: 19925b00 

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/2584234



Change-Id: I407593ef378254e504c48ea7c698b502fc2b3a29
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 389c542e 19925b00

media/codec2/sfplugin/Codec2Buffer.cpp

+47 −10
Original line number Diff line number Diff line
@@ -478,19 +478,56 @@ public: 
                mInitCheck = NO_INIT;

                return;

            case C2PlanarLayout::TYPE_RGB:

                mediaImage->mType = MediaImage2::MEDIA_IMAGE_TYPE_RGB;

                // TODO: support MediaImage layout

                switch (mClientColorFormat) {

                    case COLOR_FormatSurface:

                    case COLOR_FormatRGBFlexible:

                    case COLOR_Format24bitBGR888:

                    case COLOR_Format24bitRGB888:

                        ALOGD("Converter: accept color format "

                                "(client %d component %d) for RGB layout",

                                mClientColorFormat, mComponentColorFormat);

                        break;

                    default:

                        ALOGD("Converter: unrecognized color format "

                                "(client %d component %d) for RGB layout",

                                mClientColorFormat, mComponentColorFormat);

                mInitCheck = NO_INIT;

                // TODO: support MediaImage layout

                        mInitCheck = BAD_VALUE;

                        return;

                }

                if (layout.numPlanes != 3) {

                    ALOGD("Converter: %d planes for RGB layout", layout.numPlanes);

                    mInitCheck = BAD_VALUE;

                    return;

                }

                break;

            case C2PlanarLayout::TYPE_RGBA:

                mediaImage->mType = MediaImage2::MEDIA_IMAGE_TYPE_RGBA;

                // TODO: support MediaImage layout

                switch (mClientColorFormat) {

                    case COLOR_FormatSurface:

                    case COLOR_FormatRGBAFlexible:

                    case COLOR_Format32bitABGR8888:

                    case COLOR_Format32bitARGB8888:

                    case COLOR_Format32bitBGRA8888:

                        ALOGD("Converter: accept color format "

                                "(client %d component %d) for RGBA layout",

                                mClientColorFormat, mComponentColorFormat);

                        break;

                    default:

                        ALOGD("Converter: unrecognized color format "

                                "(client %d component %d) for RGBA layout",

                                mClientColorFormat, mComponentColorFormat);

                mInitCheck = NO_INIT;

                // TODO: support MediaImage layout

                        mInitCheck = BAD_VALUE;

                        return;

                }

                if (layout.numPlanes != 4) {

                    ALOGD("Converter: %d planes for RGBA layout", layout.numPlanes);

                    mInitCheck = BAD_VALUE;

                    return;

                }

                break;

            default:

                mediaImage->mType = MediaImage2::MEDIA_IMAGE_TYPE_UNKNOWN;

                if (layout.numPlanes == 1) {