Skip to content

[video_player_platform_interface] Add video track selection support#11474

Open
nateshmbhat wants to merge 1 commit intoflutter:mainfrom
nateshmbhat:breakout/video-track-platform-interface
Open

[video_player_platform_interface] Add video track selection support#11474
nateshmbhat wants to merge 1 commit intoflutter:mainfrom
nateshmbhat:breakout/video-track-platform-interface

Conversation

@nateshmbhat
Copy link
Copy Markdown
Contributor

Summary

Platform interface breakout PR for #10688.

  • Adds VideoTrack class with metadata fields: trackId, size (width/height), and bitrate
  • Adds getVideoTracks() method to retrieve available video quality variants
  • Adds selectVideoTrack() method to switch between quality tracks
  • Adds isVideoTrackSupportAvailable() method to check platform support

Dependency Chain

This PR is first in a series of breakout PRs:

  1. video_player_platform_interface (this PR)
  2. video_player_android (pending)
  3. video_player_avfoundation (pending)
  4. video_player + video_player_web (pending - original PR [video_player] : Add video track selection support for Android and iOS #10688 updated)

Test Plan

  • Added unit tests for VideoTrack class (constructor, equality, hashCode, toString)
  • Added unit tests for default implementations of new platform interface methods

Adds VideoTrack class and getVideoTracks(), selectVideoTrack(),
isVideoTrackSupportAvailable() methods for video track (quality) selection.

Platform interface breakout PR for flutter#10688.
Copy link
Copy Markdown

@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_platform_interface to version 6.7.0, adding a VideoTrack class and methods for video track selection: getVideoTracks, selectVideoTrack, and isVideoTrackSupportAvailable. The changes include default implementations in the platform interface and corresponding unit tests. I have no feedback to provide.

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.

1 participant