- BREAKING CHANGE: Replaced RequestPermission with RequestPermissionAsync (see yasirkula/UnityNativeGallery#343)

- BREAKING CHANGE: CheckPermission returns bool instead of Permission (see yasirkula/UnityAndroidRuntimePermissions#14)
- Updated Unity version to 2021.3.41f1
- Upgraded Texts to TextMesh Pro (closed #82)
- Configurable Enter Play Mode is supported (disabling Domain Reload works but disabling Scene Reload isn't tested)
- FileBrowser.Instance is now public for convenience
- Removed nested canvases from SimpleFileBrowserCanvas prefab to make VR integration simpler

Author: yasirkula

.github/AAR Source (Android)/java/com/yasirkula/unity/FileBrowser.java

@@ -170,20 +170,14 @@ public static int CheckPermission( Context context )
 	}
 
 	// Credit: https://github.com/Over17/UnityAndroidPermissions/blob/0dca33e40628f1f279decb67d901fd444b409cd7/src/UnityAndroidPermissions/src/main/java/com/unity3d/plugin/UnityAndroidPermissions.java
-	public static void RequestPermission( Context context, final FileBrowserPermissionReceiver permissionReceiver, final int lastCheckResult )
+	public static void RequestPermission( Context context, final FileBrowserPermissionReceiver permissionReceiver )
 	{
 		if( CheckPermission( context ) == 1 )
 		{
 			permissionReceiver.OnPermissionResult( 1 );
 			return;
 		}
 
-		if( lastCheckResult == 0 ) // If user clicked "Don't ask again" before, don't bother asking them again
-		{
-			permissionReceiver.OnPermissionResult( 0 );
-			return;
-		}
-
 		final Fragment request = new FileBrowserPermissionFragment( permissionReceiver );
 		( (Activity) context ).getFragmentManager().beginTransaction().add( 0, request ).commitAllowingStateLoss();
 	}

.github/README.md

@@ -168,25 +168,22 @@ File browser comes bundled with two premade skins in the *Skins* directory: *Lig
 - By changing *SimpleFileBrowserCanvas* prefab's *Skin* field
 - By changing the value of `FileBrowser.Skin` property from a C# script
 
-On Android, file browser requires external storage access to function properly. You can use the following function to check if we have runtime permission to access the external storage:
+On Android, file browser requires external storage access to function properly. You can use the following functions to handle runtime permissions:
 
 ```csharp
-public static FileBrowser.Permission CheckPermission();
+public delegate void PermissionCallback( Permission permission );
+
+public static bool CheckPermission();
+public static void RequestPermissionAsync( PermissionCallback callback );
 ```
 
 **FileBrowser.Permission** is an enum that can take 3 values:
 
 - **Granted**: we have the permission to access the external storage
-- **ShouldAsk**: we don't have permission yet, but we can ask the user for permission via *RequestPermission* function (see below). As long as the user doesn't select "Don't ask again" while denying the permission, ShouldAsk is returned
+- **ShouldAsk**: we don't have permission yet but we can continue asking the user for permission. As long as the user doesn't select "Don't ask again" while denying the permission, ShouldAsk is returned
 - **Denied**: we don't have permission and we can't ask the user for permission. In this case, user has to give the permission from Settings. This happens when user selects "Don't ask again" while denying the permission or when user is not allowed to give that permission (parental controls etc.)
 
-To request permission to access the external storage, use the following function:
-
-```csharp
-public static FileBrowser.Permission RequestPermission();
-```
-
-Note that FileBrowser automatically calls RequestPermission before opening a dialog. If you want, you can turn this feature off by setting **FileBrowser.AskPermissions** to *false*.
+Note that FileBrowser automatically calls RequestPermissionAsync before opening a dialog. If you want, you can turn this feature off by setting **FileBrowser.AskPermissions** to *false*.
 
 The following file manipulation functions work on all platforms (including *Storage Access Framework (SAF)* on *Android 10+*). These functions should be called with the paths returned by the FileBrowser functions only:
 

Plugins/SimpleFileBrowser/Android/FBCallbackHelper.cs

@@ -1,30 +1,49 @@
 #if UNITY_EDITOR || UNITY_ANDROID
+using System;
 using UnityEngine;
 
 namespace SimpleFileBrowser
 {
 	public class FBCallbackHelper : MonoBehaviour
 	{
-		private System.Action mainThreadAction = null;
+		private bool autoDestroyWithCallback;
+		private Action mainThreadAction = null;
 
-		private void Awake()
+		public static FBCallbackHelper Create( bool autoDestroyWithCallback )
 		{
-			DontDestroyOnLoad( gameObject );
+			FBCallbackHelper result = new GameObject( "FBCallbackHelper" ).AddComponent<FBCallbackHelper>();
+			result.autoDestroyWithCallback = autoDestroyWithCallback;
+			DontDestroyOnLoad( result.gameObject );
+			return result;
 		}
 
-		private void Update()
+		public void CallOnMainThread( Action function )
 		{
-			if( mainThreadAction != null )
+			lock( this )
 			{
-				System.Action temp = mainThreadAction;
-				mainThreadAction = null;
-				temp();
+				mainThreadAction += function;
 			}
 		}
 
-		public void CallOnMainThread( System.Action function )
+		private void Update()
 		{
-			mainThreadAction = function;
+			if( mainThreadAction != null )
+			{
+				try
+				{
+					lock( this )
+					{
+						Action temp = mainThreadAction;
+						mainThreadAction = null;
+						temp();
+					}
+				}
+				finally
+				{
+					if( autoDestroyWithCallback )
+						Destroy( gameObject );
+				}
+			}
 		}
 	}
 }

Plugins/SimpleFileBrowser/Android/FBDirectoryReceiveCallbackAndroid.cs

@@ -11,26 +11,13 @@ public class FBDirectoryReceiveCallbackAndroid : AndroidJavaProxy
 		public FBDirectoryReceiveCallbackAndroid( FileBrowser.AndroidSAFDirectoryPickCallback callback ) : base( "com.yasirkula.unity.FileBrowserDirectoryReceiver" )
 		{
 			this.callback = callback;
-			callbackHelper = new GameObject( "FBCallbackHelper" ).AddComponent<FBCallbackHelper>();
+			callbackHelper = FBCallbackHelper.Create( true );
 		}
 
 		[UnityEngine.Scripting.Preserve]
 		public void OnDirectoryPicked( string rawUri, string name )
 		{
-			callbackHelper.CallOnMainThread( () => DirectoryPickedCallback( rawUri, name ) );
-		}
-
-		private void DirectoryPickedCallback( string rawUri, string name )
-		{
-			try
-			{
-				if( callback != null )
-					callback( rawUri, name );
-			}
-			finally
-			{
-				Object.Destroy( callbackHelper.gameObject );
-			}
+			callbackHelper.CallOnMainThread( () => callback( rawUri, name ) );
 		}
 	}
 }

Plugins/SimpleFileBrowser/Android/FBPermissionCallbackAndroid.cs

@@ -1,29 +1,23 @@
 #if UNITY_EDITOR || UNITY_ANDROID
-using System.Threading;
 using UnityEngine;
 
 namespace SimpleFileBrowser
 {
 	public class FBPermissionCallbackAndroid : AndroidJavaProxy
 	{
-		private object threadLock;
-		public int Result { get; private set; }
+		private readonly FileBrowser.PermissionCallback callback;
+		private readonly FBCallbackHelper callbackHelper;
 
-		public FBPermissionCallbackAndroid( object threadLock ) : base( "com.yasirkula.unity.FileBrowserPermissionReceiver" )
+		public FBPermissionCallbackAndroid( FileBrowser.PermissionCallback callback ) : base( "com.yasirkula.unity.FileBrowserPermissionReceiver" )
 		{
-			Result = -1;
-			this.threadLock = threadLock;
+			this.callback = callback;
+			callbackHelper = FBCallbackHelper.Create( true );
 		}
 
 		[UnityEngine.Scripting.Preserve]
 		public void OnPermissionResult( int result )
 		{
-			Result = result;
-
-			lock( threadLock )
-			{
-				Monitor.Pulse( threadLock );
-			}
+			callbackHelper.CallOnMainThread( () => callback( (FileBrowser.Permission) result ) );
 		}
 	}
 }