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

Commit e3452d3f authored by Andreas Huber's avatar Andreas Huber
Browse files

Fix the media metadata retriever (tests). The native constants must match... 

Fix the media metadata retriever (tests). The native constants must match those published at the java layer.

Change-Id: Ic1efed709e4d93c61913b231a8cd3ef1074f8ca0
parent 34ec95de

include/media/MediaMetadataRetrieverInterface.h

+6 −0
Original line number Diff line number Diff line
@@ -43,6 +43,10 @@ public: 
class MediaMetadataRetrieverInterface : public MediaMetadataRetrieverBase

{

public:

    MediaMetadataRetrieverInterface()

        : mMode(0) {

    }



    virtual             ~MediaMetadataRetrieverInterface() {}



    // @param mode The intended mode of operations:
@@ -57,6 +61,8 @@ public: 
                                mode > METADATA_MODE_FRAME_CAPTURE_AND_METADATA_RETRIEVAL) {

                                return BAD_VALUE;

                            }



                            mMode = mode;

                            return NO_ERROR;

                        }

include/media/mediametadataretriever.h

+2 −2
Original line number Diff line number Diff line
@@ -67,8 +67,8 @@ enum { 
//   and meta data retrieval.$

enum {

    METADATA_MODE_NOOP                                 = 0x00,

    METADATA_MODE_FRAME_CAPTURE_ONLY                   = 0x01,

    METADATA_MODE_METADATA_RETRIEVAL_ONLY              = 0x02,

    METADATA_MODE_METADATA_RETRIEVAL_ONLY              = 0x01,

    METADATA_MODE_FRAME_CAPTURE_ONLY                   = 0x02,

    METADATA_MODE_FRAME_CAPTURE_AND_METADATA_RETRIEVAL = 0x03

};

media/libstagefright/StagefrightMetadataRetriever.cpp

+18 −0
Original line number Diff line number Diff line
@@ -206,6 +206,12 @@ static VideoFrame *extractVideoFrameWithCodecFlags( 
VideoFrame *StagefrightMetadataRetriever::captureFrame() {

    LOGV("captureFrame");



    if (0 == (mMode & METADATA_MODE_FRAME_CAPTURE_ONLY)) {

        LOGV("captureFrame disabled by mode (0x%08x)", mMode);



        return NULL;

    }



    if (mExtractor.get() == NULL) {

        LOGV("no extractor.");

        return NULL;
@@ -256,6 +262,12 @@ VideoFrame *StagefrightMetadataRetriever::captureFrame() { 
MediaAlbumArt *StagefrightMetadataRetriever::extractAlbumArt() {

    LOGV("extractAlbumArt (extractor: %s)", mExtractor.get() != NULL ? "YES" : "NO");



    if (0 == (mMode & METADATA_MODE_METADATA_RETRIEVAL_ONLY)) {

        LOGV("extractAlbumArt/metadata retrieval disabled by mode");



        return NULL;

    }



    if (mExtractor == NULL) {

        return NULL;

    }
@@ -274,6 +286,12 @@ MediaAlbumArt *StagefrightMetadataRetriever::extractAlbumArt() { 
}



const char *StagefrightMetadataRetriever::extractMetadata(int keyCode) {

    if (0 == (mMode & METADATA_MODE_METADATA_RETRIEVAL_ONLY)) {

        LOGV("extractAlbumArt/metadata retrieval disabled by mode");



        return NULL;

    }



    if (mExtractor == NULL) {

        return NULL;

    }