commit d493d39b1be60d2e7bf4f6b5a175fe6744528637
author Taehwan Kim <t_h.kim@samsung.com> Thu May 16 08:46:20 2024 +0900
committer Taehwan Kim <t_h.kim@samsung.com> Thu May 30 09:08:22 2024 +0900
tree c5d9ad0810609e1e6a2fe7dd29d5db665f8595aa
parent bbcbab33f0482ecd55d70c05692241db322ad356

webm: skip CSD for video

CodecPrivate would be handled as kMkvCodecPrivate by WebmWriter.
so, if there is CSD flag, it should be ignored.

Bug: 335614901
Bug: 342651775

Test: atest CtsMediaV2TestCases:MuxerTest
      atest CtsMediaV2TestCases:MuxerUnitTest
      atest CtsMediaV2TestCases:EncoderColorAspectsTest

Change-Id: I41f3ceafd943d21536b62b119caa4fcb66c219d2
Signed-off-by: Taehwan Kim <t_h.kim@samsung.com>

media/libstagefright/MediaMuxer.cpp

diff --git a/media/libstagefright/MediaMuxer.cpp b/media/libstagefright/MediaMuxer.cpp
index aaf7465..1008445 100644
--- a/media/libstagefright/MediaMuxer.cpp
+++ b/media/libstagefright/MediaMuxer.cpp
@@ -249,6 +249,11 @@
         sampleMetaData.setInt32(kKeyIsMuxerData, 1);
     }
 
+    if (flags & MediaCodec::BUFFER_FLAG_CODECCONFIG) {
+        sampleMetaData.setInt32(kKeyIsCodecConfig, true);
+        ALOGV("BUFFER_FLAG_CODEC_CONFIG");
+    }
+
     if (flags & MediaCodec::BUFFER_FLAG_EOS) {
         sampleMetaData.setInt32(kKeyIsEndOfStream, 1);
         ALOGV("BUFFER_FLAG_EOS");

media/libstagefright/webm/WebmFrameThread.cpp

diff --git a/media/libstagefright/webm/WebmFrameThread.cpp b/media/libstagefright/webm/WebmFrameThread.cpp
index 7d1442b..e20a08d 100644
--- a/media/libstagefright/webm/WebmFrameThread.cpp
+++ b/media/libstagefright/webm/WebmFrameThread.cpp
@@ -354,6 +354,17 @@
         }
 
         MetaDataBase &md = buffer->meta_data();
+
+        if (mType == kVideoType) {
+            int32_t isCodecConfig = 0;
+            if (md.findInt32(kKeyIsCodecConfig, &isCodecConfig) && isCodecConfig) {
+                ALOGI("ignoring CSD for video track");
+                buffer->release();
+                buffer = NULL;
+                continue;
+            }
+        }
+
         CHECK(md.findInt64(kKeyTime, &timestampUs));
         if (mStartTimeUs == kUninitialized) {
             mStartTimeUs = timestampUs;