commit 9540d81f926a92b2c44618c212b9efff359dc090
author Rajneesh Chowdury <rajneeshc@google.com> Mon Jun 27 14:16:59 2011 -0700
committer Rajneesh Chowdury <rajneeshc@google.com> Mon Jun 27 14:16:59 2011 -0700
tree f07b62642a27b36bfc0a511af5845b85b202c875
parent efc2e4255b8565f5ad6ef8d1cf0e2695e74190d4

Fix for issue 4949999 getFormat() returns NULL

Change-Id: Ie534e2660b0de2c3ac697b4e1f61e3aaf2c4cffe

libvideoeditor/vss/stagefrightshells/src/VideoEditorVideoEncoder.cpp

diff --git a/libvideoeditor/vss/stagefrightshells/src/VideoEditorVideoEncoder.cpp b/libvideoeditor/vss/stagefrightshells/src/VideoEditorVideoEncoder.cpp
index a05f48a..257ad83 100755
--- a/libvideoeditor/vss/stagefrightshells/src/VideoEditorVideoEncoder.cpp
+++ b/libvideoeditor/vss/stagefrightshells/src/VideoEditorVideoEncoder.cpp
@@ -71,7 +71,8 @@
 
 struct VideoEditorVideoEncoderSource : public MediaSource {
     public:
-        static sp<VideoEditorVideoEncoderSource> Create();
+        static sp<VideoEditorVideoEncoderSource> Create(
+            const sp<MetaData> &format);
         virtual status_t start(MetaData *params = NULL);
         virtual status_t stop();
         virtual sp<MetaData> getFormat();
@@ -92,27 +93,31 @@
             STARTED,
             ERROR
         };
-        VideoEditorVideoEncoderSource();
+        VideoEditorVideoEncoderSource(const sp<MetaData> &format);
         MediaBufferChain* mFirstBufferLink;
         MediaBufferChain* mLastBufferLink;
         int32_t           mNbBuffer;
         bool              mIsEOS;
         State             mState;
+        sp<MetaData>      mEncFormat;
 };
 
-sp<VideoEditorVideoEncoderSource> VideoEditorVideoEncoderSource::Create() {
+sp<VideoEditorVideoEncoderSource> VideoEditorVideoEncoderSource::Create(
+    const sp<MetaData> &format) {
 
     sp<VideoEditorVideoEncoderSource> aSource =
-        new VideoEditorVideoEncoderSource();
+        new VideoEditorVideoEncoderSource(format);
     return aSource;
 }
 
-VideoEditorVideoEncoderSource::VideoEditorVideoEncoderSource():
+VideoEditorVideoEncoderSource::VideoEditorVideoEncoderSource(
+    const sp<MetaData> &format):
         mFirstBufferLink(NULL),
         mLastBufferLink(NULL),
         mNbBuffer(0),
         mIsEOS(false),
-        mState(CREATED) {
+        mState(CREATED),
+        mEncFormat(format) {
     LOGV("VideoEditorVideoEncoderSource::VideoEditorVideoEncoderSource");
 }
 
@@ -170,8 +175,8 @@
 
 sp<MetaData> VideoEditorVideoEncoderSource::getFormat() {
 
-    LOGW("VideoEditorVideoEncoderSource::getFormat:THIS IS NOT IMPLEMENTED");
-    return NULL;
+    LOGV("VideoEditorVideoEncoderSource::getFormat");
+    return mEncFormat;
 }
 
 status_t VideoEditorVideoEncoderSource::read(MediaBuffer **buffer,
@@ -308,7 +313,7 @@
     VIDEOEDITOR_CHECK(CREATED == pEncoderContext->mState, M4ERR_STATE);
 
     // Create the encoder source
-    encoderSource = VideoEditorVideoEncoderSource::Create();
+    encoderSource = VideoEditorVideoEncoderSource::Create(metaData);
     VIDEOEDITOR_CHECK(NULL != encoderSource.get(), M4ERR_STATE);
 
     // Connect to the OMX client
@@ -636,7 +641,8 @@
 #endif /* VIDEOEDITOR_ENCODER_GET_DSI_AT_CREATION */
 
     // Create the encoder source
-    pEncoderContext->mEncoderSource = VideoEditorVideoEncoderSource::Create();
+    pEncoderContext->mEncoderSource = VideoEditorVideoEncoderSource::Create(
+        encoderMetadata);
     VIDEOEDITOR_CHECK(
         NULL != pEncoderContext->mEncoderSource.get(), M4ERR_STATE);