Add constructor VideoPlayerController.file() (flutter#493)

Add constructor VideoPlayerController.file

Author: sigurdm

packages/video_player/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.5.0
+
+* Added the constructor `VideoPlayerController.file`.
+* **Breaking change**. Changed `VideoPlayerController.isNetwork` to
+  an enum `VideoPlayerController.dataSourceType`.
+
 ## 0.4.1
 
 * Updated Flutter SDK constraint to reflect the changes in v0.4.0.

packages/video_player/lib/video_player.dart

@@ -3,6 +3,7 @@
 // found in the LICENSE file.
 
 import 'dart:async';
+import 'dart:io';
 
 import 'package:flutter/services.dart';
 import 'package:flutter/material.dart';
@@ -120,6 +121,8 @@ class VideoPlayerValue {
   }
 }
 
+enum DataSourceType { asset, network, file }
+
 /// Controls a platform video player, and provides updates when the state is
 /// changing.
 ///
@@ -133,7 +136,11 @@ class VideoPlayerValue {
 class VideoPlayerController extends ValueNotifier<VideoPlayerValue> {
   int _textureId;
   final String dataSource;
-  final bool isNetwork;
+
+  /// Describes the type of data source this [VideoPlayerController]
+  /// is constructed with.
+  final DataSourceType dataSourceType;
+
   String package;
   Timer timer;
   bool isDisposed = false;
@@ -147,7 +154,7 @@ class VideoPlayerController extends ValueNotifier<VideoPlayerValue> {
   /// null. The [package] argument must be non-null when the asset comes from a
   /// package and null otherwise.
   VideoPlayerController.asset(this.dataSource, {this.package})
-      : isNetwork = false,
+      : dataSourceType = DataSourceType.asset,
         super(new VideoPlayerValue(duration: null));
 
   /// Constructs a [VideoPlayerController] playing a video from obtained from
@@ -156,18 +163,39 @@ class VideoPlayerController extends ValueNotifier<VideoPlayerValue> {
   /// The URI for the video is given by the [dataSource] argument and must not be
   /// null.
   VideoPlayerController.network(this.dataSource)
-      : isNetwork = true,
+      : dataSourceType = DataSourceType.network,
+        super(new VideoPlayerValue(duration: null));
+
+  /// Constructs a [VideoPlayerController] playing a video from a file.
+  ///
+  /// This will load the file from the file-URI given by:
+  /// `'file://${file.path}'`.
+  VideoPlayerController.file(File file)
+      : dataSource = 'file://${file.path}',
+        dataSourceType = DataSourceType.file,
         super(new VideoPlayerValue(duration: null));
 
   Future<Null> initialize() async {
     _lifeCycleObserver = new _VideoAppLifeCycleObserver(this);
     _lifeCycleObserver.initialize();
     _creatingCompleter = new Completer<Null>();
+    Map<dynamic, dynamic> dataSourceDescription;
+    switch (dataSourceType) {
+      case DataSourceType.asset:
+        dataSourceDescription = <String, dynamic>{
+          'asset': dataSource,
+          'package': package
+        };
+        break;
+      case DataSourceType.network:
+        dataSourceDescription = <String, dynamic>{'uri': dataSource};
+        break;
+      case DataSourceType.file:
+        dataSourceDescription = <String, dynamic>{'uri': dataSource};
+    }
     final Map<dynamic, dynamic> response = await _channel.invokeMethod(
       'create',
-      isNetwork
-          ? <String, dynamic>{'uri': dataSource}
-          : <String, dynamic>{'asset': dataSource, 'package': package},
+      dataSourceDescription,
     );
     _textureId = response['textureId'];
     _creatingCompleter.complete(null);

packages/video_player/pubspec.yaml

@@ -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.0
 homepage: https://github.com/flutter/plugins/tree/master/packages/video_player
 
 flutter: