910995 - [Camera] Video file is not properly closed when phone is restarted while recording

Status: RESOLVED WONTFIX

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

Description

[sync-1]

Firefox os  v1.1
 Mozilla build ID:20130826041201
 
 DEFECT DESCRIPTION:
 Restart the phone during recording,video recording is not saved
 
 REPRODUCING PROCEDURES:
 1.Launch Camera app
 2.Switch to camcorder
 3.Record a video
 4.Restart the phone during recording-->Video not saved-->ko
 
 EXPECTED BEHAVIOUR:
 ko:Restart the phone during recording,video is saved
 
 ASSOCIATE SPECIFICATION:
 
 TEST PLAN REFERENCE:
 
 TOOLS AND PLATFORMS USED:
 
 USER IMPACT:
 Mid
 
 REPRODUCING RATE:
 5/5
 For FT PR, Please list reference mobile's behavior:

Comment 1

[saminath]

I tested above steps. It creates the Corrupted video(VID_0001.3gp) file on the DCIM/100MZLLA/ folder. 

In the Normal scenerio,  PowerManagerservice invoked and correspanding destructor executed.

+++++++++++++++++++++++
I/PowerManagerService(  178): Call to virtual nsresult mozilla::dom::power::PowerManagerService::PowerOff(). The JS stack is:
I/PowerManagerService(  178): 0 sm_actualPowerOff(    <Failed to get 'arguments' while inspecting stack frame>
....
E/GonkCameraControl(  906):  Release Hardware Impl  << This is called from GonkCameraControl Destructor
I/Gecko   (  606):
....
++++++++++++++++++++

But While CameraRecorder, PowerManagerService is unable to start.

Comment 2

[Dylan Oliver [:doliver]]

triage: this would be a good problem to fix but will not block release.

Comment 3

[Wilson Page [:wilsonpage]]

This is still reproduceable.

Comment 4

[Diego Marcos [:dmarcos]]

We could stop video recording when the user long presses the power button to get into the menu to restart / power off the device. Is there currently any notification we can listen to?

Comment 5

[Tim Guan-tin Chien [:timdream] (please needinfo)]

There aren't any notifications to app for that menu. Also, the menu is not entire opaque, so we shouldn't be setting the foreground app to non-visible either.

I think the proper fix of this bug is to ensure the app got a onunload events before the phone actually powers off, but I am not sure the Gecko support that or not -- are we getting onunload event when the user kills the app from the cards view?

Comment 7

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

Hema, this bug is marked P1. Is it really?

Comment 9

[Harry]

Added my vote. Apps should be able to get a beforeunload before the app is terminated for any reason. It is necessary to ensure they deal with unsaved data and terminate cleanly.

Comment 10

[BMO Automation]

Firefox OS is not being worked on