Updated platform interface to have recording methods return XFile instances

packages/camera/camera_platform_interface/lib/src/platform_interface/camera_platform.dart

@@ -4,6 +4,7 @@
 
 import 'dart:async';
 
+import 'package:file_selector_platform_interface/file_selector_platform_interface.dart';
 import 'package:flutter/widgets.dart';
 import 'package:plugin_platform_interface/plugin_platform_interface.dart';
 import 'package:camera_platform_interface/src/method_channel/method_channel_camera.dart';
@@ -66,8 +67,9 @@ abstract class CameraPlatform extends PlatformInterface {
     throw UnimplementedError('onCameraError() is not implemented.');
   }
 
-  /// Captures an image and saves it to [path].
-  Future<void> takePicture(int cameraId, String path) {
+  /// Captures an image and returns the file where it was saved.
+  /// If no [file] parameter is provided, the file returned will be in a temporary location.
+  Future<XFile> takePicture(int cameraId, { XFile file }) {
     throw UnimplementedError('takePicture() is not implemented.');
   }
 
@@ -76,20 +78,19 @@ abstract class CameraPlatform extends PlatformInterface {
     throw UnimplementedError('prepareForVideoRecording() is not implemented.');
   }
 
-  /// Start a video recording and save the file to [path].
+  /// Starts a video recording
   ///
-  /// A path can for example be obtained using
-  /// [path_provider](https://pub.dartlang.org/packages/path_provider).
+  /// If no [file] parameter is provided, the recording will be saved to a new file in a temporary location.
   ///
   /// The file is written on the flight as the video is being recorded.
-  /// If a file already exists at the provided path an error will be thrown.
-  /// The file can be read as soon as [stopVideoRecording] returns.
-  Future<void> startVideoRecording(int cameraId, String path) {
+  /// If a file already exists at the path for the provided file instance, an error will be thrown.
+  /// The file can be read as soon as [stopVideoRecording] returns it.
+  Future<XFile> startVideoRecording(int cameraId, { XFile file }) {
     throw UnimplementedError('startVideoRecording() is not implemented.');
   }
 
-  /// Stop the video recording.
-  Future<void> stopVideoRecording(int cameraId) {
+  /// Stops the video recording and returns the file where it was saved.
+  Future<XFile> stopVideoRecording(int cameraId) {
     throw UnimplementedError('stopVideoRecording() is not implemented.');
   }
 

packages/camera/camera_platform_interface/pubspec.yaml

@@ -10,6 +10,7 @@ dependencies:
     sdk: flutter
   meta: ^1.0.5
   plugin_platform_interface: ^1.0.1
+  file_selector_platform_interface: ^1.0.0
 
 dev_dependencies:
   flutter_test:

packages/camera/camera_platform_interface/test/camera_platform_interface_test.dart

@@ -156,7 +156,7 @@ void main() {
 
       // Act & Assert
       expect(
-        () => cameraPlatform.startVideoRecording(1, null),
+        () => cameraPlatform.startVideoRecording(1),
         throwsUnimplementedError,
       );
     });
@@ -182,7 +182,7 @@ void main() {
 
       // Act & Assert
       expect(
-        () => cameraPlatform.takePicture(1, null),
+        () => cameraPlatform.takePicture(1),
         throwsUnimplementedError,
       );
     });