978970 - [Camera] Refactor to utilize new start/stop recording events

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::Camera, defect)

Description

[Justin D'Arcangelo [:justindarc]]

New "start" and "stop" recording events were recently landed in Gecko for the CameraControl API (see Bug 977313). The current Camera implementation in Gaia relies on an ugly hack for listening for the latest video file written to the filesystem for knowing when recording has stopped. Changing the code to utilize the newly-added events will increase stability of the Camera app.

Comment 1

[Mike Habicher [:mikeh] (high bugzilla latency)]

Did this ever wind up landing (in another bug)?

Comment 2

[Justin D'Arcangelo [:justindarc]]

(In reply to Mike Habicher [:mikeh] from comment #1)
> Did this ever wind up landing (in another bug)?

No, because the "stop" event that we were going to rely on wasn't sufficient enough to be an indicator that the video file was done being written. Therefore, we still need an ugly hack to listen for DeviceStorage to know when our video file is ready.

Comment 3

[Justin D'Arcangelo [:justindarc]]

In order for the recording `stop` event to be useful to the Camera app, we need it to pass us the path of the recorded video file.

Comment 4

[Justin D'Arcangelo [:justindarc]]

Closing this as this has already been done for a while now.