-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Add constructor VideoPlayerController.file() #493
Add constructor VideoPlayerController.file() #493
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
packages/video_player/pubspec.yaml
Outdated
@@ -2,7 +2,7 @@ name: video_player | |||
description: Flutter plugin for displaying inline video with other Flutter | |||
widgets on Android and iOS. | |||
author: Flutter Team <flutter-dev@googlegroups.com> | |||
version: 0.4.1 | |||
version: 0.5.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
0.5.0 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Done
@@ -133,7 +140,10 @@ class VideoPlayerValue { | |||
class VideoPlayerController extends ValueNotifier<VideoPlayerValue> { | |||
int _textureId; | |||
final String dataSource; | |||
final bool isNetwork; | |||
|
|||
/// Describes the type of data source this is constructed with. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: this -> this VideoPlayerController
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review!
@@ -133,7 +140,10 @@ class VideoPlayerValue { | |||
class VideoPlayerController extends ValueNotifier<VideoPlayerValue> { | |||
int _textureId; | |||
final String dataSource; | |||
final bool isNetwork; | |||
|
|||
/// Describes the type of data source this is constructed with. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
packages/video_player/pubspec.yaml
Outdated
@@ -2,7 +2,7 @@ name: video_player | |||
description: Flutter plugin for displaying inline video with other Flutter | |||
widgets on Android and iOS. | |||
author: Flutter Team <flutter-dev@googlegroups.com> | |||
version: 0.4.1 | |||
version: 0.5.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Done
Add constructor VideoPlayerController.file
This fixes flutter/flutter#16444.