From fac262516e3498414721af2f6abc3c8b8176f99e Mon Sep 17 00:00:00 2001 From: xiaomings Date: Mon, 29 Apr 2024 10:11:19 -0700 Subject: [PATCH] [android] Fix low fps checking in video_decoder.cc (#3086) The mime attribute "framerate" is used to specify the frame rate of the video, and the previous implementation incorrectly used "fps" as the mime attribute name. b/286121490 Change-Id: Iaec70f33ab625224e8a31eba96db0bea063f0a8a --- starboard/android/shared/video_decoder.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/starboard/android/shared/video_decoder.cc b/starboard/android/shared/video_decoder.cc index 3cdcc59e56ff..fd509ad51631 100644 --- a/starboard/android/shared/video_decoder.cc +++ b/starboard/android/shared/video_decoder.cc @@ -87,7 +87,7 @@ bool IsSoftwareDecodeRequired(const std::string& max_video_capabilities) { bool is_low_resolution = mime_type.GetParamIntValue("width", 1920) <= 432 && mime_type.GetParamIntValue("height", 1080) <= 240; - bool is_low_fps = mime_type.GetParamIntValue("fps", 30) <= 15; + bool is_low_fps = mime_type.GetParamIntValue("framerate", 30) <= 15; if (is_low_resolution && is_low_fps) { // Workaround to be compatible with existing backend implementation.