Latest version -- NativeGallery.SaveVideoToGallery('filePath') fails, but succeeds if you use byte[]

[Saif0089]

Description of the bug

Saving an existing .mp4 file into gallery fails.

NativeGallery.SaveVideoToGallery(recordingMP4FilePath, _album, videoFileName,
(success, path) =>
{
    Debug.Log($"VideoRecorder => Video saved to gallery in {_album}");
    
    OnSavedToGallery?.Invoke(videoFileName);
    
    File.Delete(recordingMP4FilePath);
});

Reproduction steps

Try saving an existing video file into Gallery.

Platform specs

• Unity version: 6000.1.10f1
• Platform: iOS
• Device: iPhone 15 Pro, iOS 18.6.2
• Asset was up-to-date, was installed using OpenUPM, v1.9.1*

Additional info

[yasirkula]

Looking at FileNotFoundException, I'd suggest that this is a NatCorder issue. Can you play the video using Handheld.PlayFullScreenMovie? Does File.Exists(filePath) return true?

[Saif0089]

Yes, File.Exists(filePath) does return true.

And the FileNotFoundException exception disappears if I use fileData as byte[] instead of feeding a recordingMP4FilePath into SaveVideoToGallery.

Like this...

NativeGallery.SaveVideoToGallery(File.ReadAllBytes(recordingMP4FilePath), _album, videoFileName,
						(success, path) =>
					{
						Debug.Log($"VideoRecorder => Video saved to gallery in {_album}");

						OnSavedToGallery?.Invoke(videoFileName);

						File.Delete(recordingMP4FilePath);
					});

[Saif0089]

Using recordingMP4FilePath was working fine in 1.8.0.

[yasirkula]

It's still possible that recordingMP4FilePath is deleted in a few frames if you don't copy/move it. For example, can you cache its value and in Update, print File.Exists(recordingMP4FilePath) if recordingMP4FilePath isn't null? I expect it to be true for only a few frames.

In v1.9.0, synchronous permission system was deleted because it was problematic and wasn't a documented or recommended feature of Android/iOS permission system. SaveVideoToGallery used to call File.Copy in the same frame but now it's called asynchronously. If my theory about NatCorder is correct, then you'll need to continue using File.ReadAllBytes approach.