VideoRecordEvent.Finalize.ERROR_INSUFFICIENT_STORAGE is never called [356399842]

Fixed

Bug

Status Update

No update yet.

Description

wq...@gmail.com

created issue #1

Jul 31, 2024 06:42AM

When recording video via Recorder, I try to handle an error when there is not enough memory for the recorded video. The documentation has an example with this case (

https://developer.android.com/reference/androidx/camera/video/VideoRecordEvent.Finalize).
But according to the androidx.camera.video.Recorder code, the VideoRecordEvent.Finalize.ERROR_INSUFFICIENT_STORAGE event is never called. In real use, I receive VideoRecordEvent.Finalize.ERROR_INVALID_OUTPUT_OPTIONS in VideoRecordEvent with the error java.io.FileNotFoundException: /data/user/0/**/file_name.mp4: open failed: ENOSPC (No space left on device).
Also, if the space runs out during recording, then VideoRecordEvent stops receiving any events, including VideoRecordEvent.Status

Comments

wu...@google.com <wu...@google.com> Jul 31, 2024 07:16AM

Reassigned to le...@google.com.

le...@google.com <le...@google.com> #2Aug 2, 2024 02:34AM

Hi Ed, Thank you so much for these suggestions. I've been reviewing them and merging them in. Hopefully it should be live. I've included a thank you note too in the article.

le...@google.com <le...@google.com> Sep 5, 2024 07:12AM

Accepted by le...@google.com.

ap...@google.com <ap...@google.com> #3Sep 24, 2024 08:03AM

Great! Thanks a lot, I'll look for the live updates soon!

ap...@google.com <ap...@google.com> #4Sep 24, 2024 08:03AM

Project: platform/frameworks/support
Branch: androidx-main

commit 731951d4fd675c17c84465af92557247e4109777
Author: leo huang <leohuang@google.com>
Date: Wed Sep 04 18:50:57 2024

Catch storage full exceptions and trigger insufficient storage error while video recording

Relnote: "Fixed the VideoRecordEvent.Finalize.ERROR_INSUFFICIENT_STORAGE event is not triggered when recording a video and there is insufficient storage space available."

Bug: 356399842
Test: manully testing
Change-Id: Ia5b4f435e387b690fa4e7bb13e747d636a2d72b0

M camera/camera-video/src/main/java/androidx/camera/video/Recorder.java
M camera/camera-video/src/main/java/androidx/camera/video/internal/utils/StorageUtil.kt

https://android-review.googlesource.com/3259900

le...@google.com <le...@google.com> #5Sep 24, 2024 08:04AM

Marked as fixed.

The solution will be included in 1.5.0-alpha02

Message last modified on Sep 24, 2024 08:04AM

le...@google.com <le...@google.com> Sep 24, 2024 04:07PM

Status: Assigned (reopened)

le...@google.com <le...@google.com> #6Sep 26, 2024 04:03AM

Accepted by le...@google.com.

1.5.0-alpha02 is on the way. We will postpone this fix on 1.5.0-alpha03.

ap...@google.com <ap...@google.com> #7Oct 15, 2024 01:03AM

Project: platform/frameworks/support
Branch: androidx-main
Author: leo huang <leohuang@google.com>
Link: https://android-review.googlesource.com/3281674

Add insufficient storage error trigger for video recording

Expand for full commit details

Add insufficient storage error trigger for video recording

 - A new setting has been added to specify the required free storage, defaulting to 50 MB. When available storage falls below this value, an "insufficient storage" error will be triggered.

 - For efficiency and simplicity, storage size will not be periodically polled during recording. We assume storage is only consumed by the recording itself, and it will only refresh the available storage size once the recorded data exceeds the required free storage threshold.

 - See go/camerax_videocapture_insufficient_storage

Bug: 356399842
Test: RecorderTest
Change-Id: Ib5bf3a44353acf264d6e68b5574191ef72dbd8e4

Files:

M camera/camera-video/src/androidTest/java/androidx/camera/video/RecorderTest.kt
M camera/camera-video/src/main/java/androidx/camera/video/Recorder.java
A camera/camera-video/src/main/java/androidx/camera/video/internal/OutputStorage.kt
A camera/camera-video/src/main/java/androidx/camera/video/internal/OutputStorageImpl.kt
A camera/camera-video/src/main/java/androidx/camera/video/internal/utils/StorageUtil.kt

Hash: 5253ac2e2ec412d8ca3edcb813717dc73960134b
Date: Fri Aug 30 14:28:18 2024

ap...@google.com <ap...@google.com> #8Oct 15, 2024 01:03AM

Project: platform/frameworks/support
Branch: androidx-main
Author: leo huang <leohuang@google.com>
Link: https://android-review.googlesource.com/3281675

Catch storage full exceptions and trigger insufficient storage error while video recording

Expand for full commit details

Catch storage full exceptions and trigger insufficient storage error while video recording

See go/camerax_videocapture_insufficient_storage

Relnote: "Fixed the VideoRecordEvent.Finalize.ERROR_INSUFFICIENT_STORAGE event is not triggered when recording a video and there is insufficient storage space available."

Bug: 356399842
Test: manully testing
Change-Id: I35779fd7edbbeaff77cdf917eea1b377dc3e2151

Files:

M camera/camera-video/src/main/java/androidx/camera/video/Recorder.java
M camera/camera-video/src/main/java/androidx/camera/video/internal/utils/StorageUtil.kt

Hash: 55bf8082aa50ebc33ff676801b9bd153ff6b638e
Date: Wed Sep 04 18:50:57 2024

le...@google.com <le...@google.com> Oct 24, 2024 08:22AM

Marked as fixed.

pr...@google.com <pr...@google.com> #9Oct 30, 2024 05:37PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.camera:camera-video:1.5.0-alpha03

01...@gmail.com <01...@gmail.com> Nov 17, 2024 07:30PM

deleted

Restricted

0 B