[image_picker] return error in the event that permissions are not granted

packages/image_picker/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.6.0+15
+
+* Android: throws an exception when permissions denied instead of ignoring.
+
 ## 0.6.0+14
 
 * Fix typo in README.

packages/image_picker/android/src/main/java/io/flutter/plugins/imagepicker/ImagePickerDelegate.java

@@ -408,7 +408,16 @@ public boolean onRequestPermissionsResult(
     }
 
     if (!permissionGranted) {
-      finishWithSuccess(null);
+      switch (requestCode) {
+        case REQUEST_EXTERNAL_IMAGE_STORAGE_PERMISSION:
+        case REQUEST_EXTERNAL_VIDEO_STORAGE_PERMISSION:
+          finishWithError("photo_access_denied", "The user did not allow photo access.");
+          break;
+        case REQUEST_CAMERA_IMAGE_PERMISSION:
+        case REQUEST_CAMERA_VIDEO_PERMISSION:
+          finishWithError("camera_access_denied", "The user did not allow camera access.");
+          break;
+      }
     }
 
     return true;

packages/image_picker/example/android/app/src/test/java/io/flutter/plugins/imagepicker/ImagePickerDelegateTest.java

@@ -184,15 +184,15 @@ public void takeImageWithCamera_WhenCameraPermissionNotPresent_RequestsForPermis
 
   @Test
   public void
-      onRequestPermissionsResult_WhenReadExternalStoragePermissionDenied_FinishesWithNull() {
+      onRequestPermissionsResult_WhenReadExternalStoragePermissionDenied_FinishesWithError() {
     ImagePickerDelegate delegate = createDelegateWithPendingResultAndMethodCall();
 
     delegate.onRequestPermissionsResult(
         ImagePickerDelegate.REQUEST_EXTERNAL_IMAGE_STORAGE_PERMISSION,
         new String[] {Manifest.permission.READ_EXTERNAL_STORAGE},
         new int[] {PackageManager.PERMISSION_DENIED});
 
-    verify(mockResult).success(null);
+    verify(mockResult).error("photo_access_denied", "The user did not allow photo access.", null);
     verifyNoMoreInteractions(mockResult);
   }
 
@@ -227,15 +227,15 @@ public void takeImageWithCamera_WhenCameraPermissionNotPresent_RequestsForPermis
   }
 
   @Test
-  public void onRequestPermissionsResult_WhenCameraPermissionDenied_FinishesWithNull() {
+  public void onRequestPermissionsResult_WhenCameraPermissionDenied_FinishesWithError() {
     ImagePickerDelegate delegate = createDelegateWithPendingResultAndMethodCall();
 
     delegate.onRequestPermissionsResult(
         ImagePickerDelegate.REQUEST_CAMERA_IMAGE_PERMISSION,
         new String[] {Manifest.permission.CAMERA},
         new int[] {PackageManager.PERMISSION_DENIED});
 
-    verify(mockResult).success(null);
+    verify(mockResult).error("camera_access_denied", "The user did not allow camera access.", null);
     verifyNoMoreInteractions(mockResult);
   }
 

packages/image_picker/pubspec.yaml

@@ -5,7 +5,8 @@ authors:
   - Flutter Team <flutter-dev@googlegroups.com>
   - Rhodes Davis Jr. <rody.davis.jr@gmail.com>
 homepage: https://github.com/flutter/plugins/tree/master/packages/image_picker
-version: 0.6.0+14
+
+version: 0.6.0+15
 
 flutter:
   plugin: