[TTVD] Skip setting MJPEG decoder input type 29/318529/3
authorJakub Gajownik <j.gajownik2@samsung.com>
Mon, 30 Sep 2024 08:19:14 +0000 (10:19 +0200)
committerBot Blink <blinkbot@samsung.com>
Tue, 1 Oct 2024 18:28:10 +0000 (18:28 +0000)
Video decoder input type is not used in implementation
of MJPEG video decoder and on newer versions it will report
error when trying to do so. We should skip setting it for
this specific codec.

Bug: https://jira-eu.sec.samsung.net/browse/VDGAME-589
Change-Id: Iccf1582d70436ebc4f490337e1168ddb557a0990
Signed-off-by: Jakub Gajownik <j.gajownik2@samsung.com>
media/filters/tizen/omx/omx_facade_video.cc

index 40e9b928c854ed262e5b5383e90b7898b4f32a78..68a9a13fdef2ce03cb873eb3f7136dd534362f09 100644 (file)
@@ -199,6 +199,12 @@ void OmxFacadeVideo::OnOmxEvent(OMX_HANDLETYPE hComponent,
 
 bool OmxFacadeVideo::SetComponentConfiguration(
     const std::string& component_name) {
+  if (codec_ == MediaVideoCodec::kCodecMJPEG) {
+    // Setting video decoder input param is treated as error with MJPEG decoder,
+    // so skip it to avoid errors.
+    return true;
+  }
+
   OMX_VIDEO_PARAM_DECODERINPUTTYPE param{};
   if (low_delay_) {
     param.nVideoDecodingType = OMX_Video_Decoding_Clip;