refactor(video_player): replace track ID with group/track indices

- Changed VideoAudioTrack to use groupIndex and trackIndex instead of a single string ID
- Updated selectAudioTrack to accept VideoAudioTrack object instead of string trackId
- Modified UI to display track indices in format "Track {groupIndex}_{trackIndex}"

Author: nateshmbhat

packages/video_player/video_player/example/lib/audio_tracks_demo.dart

@@ -106,24 +106,28 @@ class _AudioTracksDemoState extends State<AudioTracksDemo> {
     }
   }
 
-  Future<void> _selectAudioTrack(String trackId) async {
+  Future<void> _selectAudioTrack(VideoAudioTrack track) async {
     final VideoPlayerController? controller = _controller;
     if (controller == null) {
       return;
     }
 
     try {
-      await controller.selectAudioTrack(trackId);
+      await controller.selectAudioTrack(track);
 
       // Reload tracks to update selection status
       await _loadAudioTracks();
 
       if (!mounted) {
         return;
       }
-      ScaffoldMessenger.of(
-        context,
-      ).showSnackBar(SnackBar(content: Text('Selected audio track: $trackId')));
+      ScaffoldMessenger.of(context).showSnackBar(
+        SnackBar(
+          content: Text(
+            'Selected audio track: ${track.label.isNotEmpty ? track.label : "Track ${track.groupIndex}_${track.trackIndex}"}',
+          ),
+        ),
+      );
     } catch (e) {
       if (!mounted) {
         return;
@@ -348,15 +352,17 @@ class _AudioTracksDemoState extends State<AudioTracksDemo> {
           ),
         ),
         title: Text(
-          track.label.isNotEmpty ? track.label : 'Track ${track.id}',
+          track.label.isNotEmpty
+              ? track.label
+              : 'Track ${track.groupIndex}_${track.trackIndex}',
           style: TextStyle(
             fontWeight: track.isSelected ? FontWeight.bold : FontWeight.normal,
           ),
         ),
         subtitle: Column(
           crossAxisAlignment: CrossAxisAlignment.start,
           children: <Widget>[
-            Text('ID: ${track.id}'),
+            Text('Group: ${track.groupIndex}, Track: ${track.trackIndex}'),
             Text('Language: ${track.language}'),
             if (track.codec != null) Text('Codec: ${track.codec}'),
             if (track.bitrate != null) Text('Bitrate: ${track.bitrate} bps'),
@@ -370,7 +376,7 @@ class _AudioTracksDemoState extends State<AudioTracksDemo> {
             track.isSelected
                 ? const Icon(Icons.radio_button_checked, color: Colors.green)
                 : const Icon(Icons.radio_button_unchecked),
-        onTap: track.isSelected ? null : () => _selectAudioTrack(track.id),
+        onTap: track.isSelected ? null : () => _selectAudioTrack(track),
       ),
     );
   }

packages/video_player/video_player/example/pubspec.yaml

@@ -38,8 +38,7 @@ flutter:
 # FOR TESTING AND INITIAL REVIEW ONLY. DO NOT MERGE.
 # See https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#changing-federated-plugins
 dependency_overrides:
-  video_player_android:
-    { path: ../../../../packages/video_player/video_player_android }
-  video_player_avfoundation:
-    { path: ../../../../packages/video_player/video_player_avfoundation }
-  video_player_web: { path: ../../../../packages/video_player/video_player_web }
+  video_player_android: {path: ../../../../packages/video_player/video_player_android}
+  video_player_avfoundation: {path: ../../../../packages/video_player/video_player_avfoundation}
+  video_player_platform_interface: {path: ../../../../packages/video_player/video_player_platform_interface}
+  video_player_web: {path: ../../../../packages/video_player/video_player_web}

packages/video_player/video_player/lib/video_player.dart

@@ -30,7 +30,8 @@ export 'src/closed_caption_file.dart';
 class VideoAudioTrack {
   /// Constructs an instance of [VideoAudioTrack].
   const VideoAudioTrack({
-    required this.id,
+    required this.groupIndex,
+    required this.trackIndex,
     required this.label,
     required this.language,
     required this.isSelected,
@@ -40,8 +41,11 @@ class VideoAudioTrack {
     this.codec,
   });
 
-  /// Unique identifier for the audio track.
-  final String id;
+  /// The group index of the audio track.
+  final int groupIndex;
+
+  /// The track index within the group.
+  final int trackIndex;
 
   /// Human-readable label for the track.
   final String label;
@@ -73,7 +77,8 @@ class VideoAudioTrack {
     return identical(this, other) ||
         other is VideoAudioTrack &&
             runtimeType == other.runtimeType &&
-            id == other.id &&
+            groupIndex == other.groupIndex &&
+            trackIndex == other.trackIndex &&
             label == other.label &&
             language == other.language &&
             isSelected == other.isSelected &&
@@ -85,7 +90,8 @@ class VideoAudioTrack {
 
   @override
   int get hashCode => Object.hash(
-    id,
+    groupIndex,
+    trackIndex,
     label,
     language,
     isSelected,
@@ -98,7 +104,8 @@ class VideoAudioTrack {
   @override
   String toString() =>
       'VideoAudioTrack('
-      'id: $id, '
+      'groupIndex: $groupIndex, '
+      'trackIndex: $trackIndex, '
       'label: $label, '
       'language: $language, '
       'isSelected: $isSelected, '
@@ -120,7 +127,8 @@ VideoAudioTrack _convertPlatformAudioTrack(
   platform_interface.VideoAudioTrack platformTrack,
 ) {
   return VideoAudioTrack(
-    id: platformTrack.id,
+    groupIndex: platformTrack.groupIndex,
+    trackIndex: platformTrack.trackIndex,
     label: platformTrack.label ?? 'Unknown',
     language: platformTrack.language ?? 'und',
     isSelected: platformTrack.isSelected,
@@ -945,20 +953,27 @@ class VideoPlayerController extends ValueNotifier<VideoPlayerValue> {
     return platformTracks.map(_convertPlatformAudioTrack).toList();
   }
 
-  /// Selects which audio track is chosen for playback from its [trackId]
-  ///
-  /// The [trackId] should match the ID of one of the tracks returned by
-  /// [getAudioTracks]. If the track ID is not found or invalid, the
-  /// platform may ignore the request or throw an exception.
+  /// Selects which audio track is chosen for playback.
   ///
-  /// Throws an exception if the video player is disposed or not initialized.
-  Future<void> selectAudioTrack(String trackId) async {
+  /// The [track] parameter should be one of the tracks returned by [getAudioTracks].
+  Future<void> selectAudioTrack(VideoAudioTrack track) async {
     if (_isDisposedOrNotInitialized) {
       throw Exception('VideoPlayerController is disposed or not initialized');
     }
-    // The platform implementation (e.g., Android) will wait for the track
-    // selection to complete by listening to platform-specific events
-    await _videoPlayerPlatform.selectAudioTrack(_playerId, trackId);
+    // Convert the public VideoAudioTrack to platform interface VideoAudioTrack
+    final platform_interface.VideoAudioTrack platformTrack =
+        platform_interface.VideoAudioTrack(
+          groupIndex: track.groupIndex,
+          trackIndex: track.trackIndex,
+          label: track.label,
+          language: track.language,
+          isSelected: track.isSelected,
+          bitrate: track.bitrate,
+          sampleRate: track.sampleRate,
+          channelCount: track.channelCount,
+          codec: track.codec,
+        );
+    await _videoPlayerPlatform.selectAudioTrack(_playerId, platformTrack);
   }
 
   /// Returns whether audio track selection is supported on this platform.

packages/video_player/video_player/pubspec.yaml

@@ -41,8 +41,7 @@ topics:
 # FOR TESTING AND INITIAL REVIEW ONLY. DO NOT MERGE.
 # See https://github.com/flutter/flutter/blob/master/docs/ecosystem/contributing/README.md#changing-federated-plugins
 dependency_overrides:
-  video_player_android:
-    { path: ../../../packages/video_player/video_player_android }
-  video_player_avfoundation:
-    { path: ../../../packages/video_player/video_player_avfoundation }
-  video_player_web: { path: ../../../packages/video_player/video_player_web }
+  video_player_android: {path: ../../../packages/video_player/video_player_android}
+  video_player_avfoundation: {path: ../../../packages/video_player/video_player_avfoundation}
+  video_player_platform_interface: {path: ../../../packages/video_player/video_player_platform_interface}
+  video_player_web: {path: ../../../packages/video_player/video_player_web}