@@ -12,9 +12,6 @@ import 'package:web/web.dart' as web;
1212
1313import 'src/video_player.dart' ;
1414
15- // TODO(FirentisTFW): Remove the ignore and rename parameters when adding support for platform views.
16- // ignore_for_file: avoid_renaming_method_parameters
17-
1815/// The web implementation of [VideoPlayerPlatform] .
1916///
2017/// This class implements the `package:video_player` functionality for the web.
@@ -24,21 +21,20 @@ class VideoPlayerPlugin extends VideoPlayerPlatform {
2421 VideoPlayerPlatform .instance = VideoPlayerPlugin ();
2522 }
2623
27- // Map of textureId -> VideoPlayer instances
24+ // Map of playerId -> VideoPlayer instances.
2825 final Map <int , VideoPlayer > _videoPlayers = < int , VideoPlayer > {};
2926
30- // Simulate the native "textureId".
31- int _textureCounter = 1 ;
27+ int _playerCounter = 1 ;
3228
3329 @override
3430 Future <void > init () async {
3531 return _disposeAllPlayers ();
3632 }
3733
3834 @override
39- Future <void > dispose (int textureId ) async {
40- _player (textureId ).dispose ();
41- _videoPlayers.remove (textureId );
35+ Future <void > dispose (int playerId ) async {
36+ _player (playerId ).dispose ();
37+ _videoPlayers.remove (playerId );
4238 return ;
4339 }
4440
@@ -50,8 +46,22 @@ class VideoPlayerPlugin extends VideoPlayerPlatform {
5046 }
5147
5248 @override
53- Future <int > create (DataSource dataSource) async {
54- final int textureId = _textureCounter++ ;
49+ Future <int > create (DataSource dataSource) {
50+ return createWithOptions (
51+ VideoCreationOptions (
52+ dataSource: dataSource,
53+ // Web only supports platform views.
54+ viewType: VideoViewType .platformView,
55+ ),
56+ );
57+ }
58+
59+ @override
60+ Future <int > createWithOptions (VideoCreationOptions options) async {
61+ // Parameter options.viewType is ignored because web only supports platform views.
62+
63+ final DataSource dataSource = options.dataSource;
64+ final int playerId = _playerCounter++ ;
5565
5666 late String uri;
5767 switch (dataSource.sourceType) {
@@ -75,68 +85,68 @@ class VideoPlayerPlugin extends VideoPlayerPlatform {
7585 }
7686
7787 final web.HTMLVideoElement videoElement = web.HTMLVideoElement ()
78- ..id = 'videoElement-$textureId '
88+ ..id = 'videoElement-$playerId '
7989 ..style.border = 'none'
8090 ..style.height = '100%'
8191 ..style.width = '100%' ;
8292
8393 // TODO(hterkelsen): Use initialization parameters once they are available
8494 ui_web.platformViewRegistry.registerViewFactory (
85- 'videoPlayer-$textureId ' , (int viewId) => videoElement);
95+ 'videoPlayer-$playerId ' , (int viewId) => videoElement);
8696
8797 final VideoPlayer player = VideoPlayer (videoElement: videoElement)
8898 ..initialize (
8999 src: uri,
90100 );
91101
92- _videoPlayers[textureId ] = player;
102+ _videoPlayers[playerId ] = player;
93103
94- return textureId ;
104+ return playerId ;
95105 }
96106
97107 @override
98- Future <void > setLooping (int textureId , bool looping) async {
99- return _player (textureId ).setLooping (looping);
108+ Future <void > setLooping (int playerId , bool looping) async {
109+ return _player (playerId ).setLooping (looping);
100110 }
101111
102112 @override
103- Future <void > play (int textureId ) async {
104- return _player (textureId ).play ();
113+ Future <void > play (int playerId ) async {
114+ return _player (playerId ).play ();
105115 }
106116
107117 @override
108- Future <void > pause (int textureId ) async {
109- return _player (textureId ).pause ();
118+ Future <void > pause (int playerId ) async {
119+ return _player (playerId ).pause ();
110120 }
111121
112122 @override
113- Future <void > setVolume (int textureId , double volume) async {
114- return _player (textureId ).setVolume (volume);
123+ Future <void > setVolume (int playerId , double volume) async {
124+ return _player (playerId ).setVolume (volume);
115125 }
116126
117127 @override
118- Future <void > setPlaybackSpeed (int textureId , double speed) async {
119- return _player (textureId ).setPlaybackSpeed (speed);
128+ Future <void > setPlaybackSpeed (int playerId , double speed) async {
129+ return _player (playerId ).setPlaybackSpeed (speed);
120130 }
121131
122132 @override
123- Future <void > seekTo (int textureId , Duration position) async {
124- return _player (textureId ).seekTo (position);
133+ Future <void > seekTo (int playerId , Duration position) async {
134+ return _player (playerId ).seekTo (position);
125135 }
126136
127137 @override
128- Future <Duration > getPosition (int textureId ) async {
129- return _player (textureId ).getPosition ();
138+ Future <Duration > getPosition (int playerId ) async {
139+ return _player (playerId ).getPosition ();
130140 }
131141
132142 @override
133- Stream <VideoEvent > videoEventsFor (int textureId ) {
134- return _player (textureId ).events;
143+ Stream <VideoEvent > videoEventsFor (int playerId ) {
144+ return _player (playerId ).events;
135145 }
136146
137147 @override
138- Future <void > setWebOptions (int textureId , VideoPlayerWebOptions options) {
139- return _player (textureId ).setOptions (options);
148+ Future <void > setWebOptions (int playerId , VideoPlayerWebOptions options) {
149+ return _player (playerId ).setOptions (options);
140150 }
141151
142152 // Retrieves a [VideoPlayer] by its internal `id`.
@@ -146,11 +156,11 @@ class VideoPlayerPlugin extends VideoPlayerPlatform {
146156 }
147157
148158 @override
149- Widget buildView (int textureId ) {
150- return HtmlElementView (viewType: 'videoPlayer-$textureId ' );
159+ Widget buildView (int playerId ) {
160+ return HtmlElementView (viewType: 'videoPlayer-$playerId ' );
151161 }
152162
153- /// Sets the audio mode to mix with other sources (ignored)
163+ /// Sets the audio mode to mix with other sources (ignored).
154164@override
155165 Future <void > setMixWithOthers (bool mixWithOthers) => Future <void >.value ();
156166}
0 commit comments