Skip to content

[video_player_avplay] Fixed an issue where subtitles for content with pre-roll/insesrted ads in DASH could not be rendered.#1007

Merged
xiaowei-guan merged 6 commits intoflutter-tizen:masterfrom
gin7773:subtitle_issue03
Apr 24, 2026
Merged

[video_player_avplay] Fixed an issue where subtitles for content with pre-roll/insesrted ads in DASH could not be rendered.#1007
xiaowei-guan merged 6 commits intoflutter-tizen:masterfrom
gin7773:subtitle_issue03

Conversation

@gin7773
Copy link
Copy Markdown
Contributor

@gin7773 gin7773 commented Apr 20, 2026

No description provided.

@gin7773 gin7773 marked this pull request as draft April 20, 2026 09:34
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the caption selection logic in video_player.dart to include start-time checks for both text and picture captions. A logic error was identified in the picture caption implementation where the greater-than operator was used instead of the less-than operator for the start time check, which would cause captions to be hidden incorrectly during their display window.

Comment thread packages/video_player_avplay/lib/video_player.dart Outdated
@gin7773 gin7773 marked this pull request as ready for review April 23, 2026 10:52
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the video_player_avplay package to version 0.8.8 and modifies the _currentCaptions getter to prevent subtitle rendering when the playback position is before the caption start time. Feedback identifies critical safety issues where the new conditional checks could cause runtime crashes due to null or empty list access, and notes that similar logic in other methods remains inconsistent. Additionally, a typo was found in the changelog.

Comment thread packages/video_player_avplay/lib/video_player.dart Outdated
Comment thread packages/video_player_avplay/lib/video_player.dart Outdated
Comment thread packages/video_player_avplay/CHANGELOG.md Outdated
@xiaowei-guan xiaowei-guan merged commit bea1df2 into flutter-tizen:master Apr 24, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants