Closed Bug 986763 Opened 7 years ago Closed 7 years ago

[Camera][Madai] Very short video not saved.

Categories

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

ARM
macOS
defect
Not set
normal

Tracking

(b2g-v1.4 verified, b2g-v2.0 verified)

VERIFIED FIXED
1.4 S5 (11apr)
Tracking Status
b2g-v1.4 --- verified
b2g-v2.0 --- verified

People

(Reporter: dmarcos, Assigned: ver4ffos)

References

Details

(Whiteboard: [m+])

Attachments

(2 files)

46 bytes, text/x-github-pull-request
justindarc
: review+
dmarcos
: feedback?
ver4ffos
dmarcos
: feedback?
gjyothiprasad
Details | Review
46 bytes, text/x-github-pull-request
Details | Review
When switching from recording to camera we stop recording. If the video recorded is very short (less than one second) it doesn't get saved and modal window with no message is displayed.

Steps to reproduce:

1. Open Camera
2. Switch to video
3. Start recording
4. Switch to photo quickly, only allowing to record a video shorter than half a second
Blocks: 983405
Youngjun,

Do you have someone from your team who can investigate?
Flags: needinfo?(jjoons79)
Dear Prasad and Verchaswa,

Can you take this issue?
Flags: needinfo?(ver4ffos)
Flags: needinfo?(jjoons79)
Flags: needinfo?(gjyothiprasad)
Dear Mr. Youngjun Kim,
We will take the issue.
Flags: needinfo?(ver4ffos)
Assignee: nobody → ver4ffos
Flags: needinfo?(gjyothiprasad)
Mr. Diego,

Need your opinion:
Should not we disable the switch mode button when recording? This will avoid the issue.
Why we need to enable switch mode button, when recording is going on? The scenario.
Flags: needinfo?(dmarcos)
I created a simple patch to hide the switch button while recording video
Flags: needinfo?(dmarcos)
Attachment #8396264 - Flags: review?(dflanagan)
Attachment #8396264 - Flags: feedback?(ver4ffos)
Attachment #8396264 - Flags: feedback?(gjyothiprasad)
Patch is tested and it is working fine.
Comment on attachment 8396264 [details] [review]
Pull Request on camera-new-features

Transferring review to Justin
Attachment #8396264 - Flags: review?(dflanagan) → review?(jdarcangelo)
Comment on attachment 8396264 [details] [review]
Pull Request on camera-new-features

Diego: This looks good, but needs rebased for master now that camera-new-features has landed.
Attachment #8396264 - Flags: review?(jdarcangelo) → review+
Before marking this fix, please ensure this case works...

> 
> Here are the steps from Qualcomm
> 
>     Go to camera app, and switch to camcorder
> 
> ·         Start recording the video
> 
> ·         Switch back to camera without stopping the recording.
> 
> ·         Recording still continues and you are able to take picture as well.
>
Whiteboard: [m+]
Attachment #8396264 - Attachment description: Pull Request on camera new features → Pull Request on master
Attachment #8396264 - Attachment description: Pull Request on master → Pull Request on camera-new-features
Attached file Pull Request on master
Rebased patch on top of master. Carrying r+ over
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Bulk edit for camera bugs.

If earlier comments do not show how this bug landed to master, it probably landed as part of https://github.com/mozilla-b2g/gaia/pull/17599 which merged the camera-new-features branch into master.

This bug was uplifted from master to v1.4 as part of https://github.com/mozilla-b2g/gaia/commit/a8190d08e61316a86bba572ba8d894d081a20530
Target Milestone: --- → 1.4 S5 (11apr)
Verified the bug is fixed, switch mode is disabled during recording


1.4 Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140408000202
Gaia: 26983f356ecb1bcf30e862d334b5de790071803e
Gecko: 70b076fc7558
Version: 30.0a2
Firmware Version: v1.2-device.cfg

1.5 Environmental Variables:
Device: Buri Master 
BuildID: 20140408040204
Gaia: 1958454595b1fa0e061f0652ae965629993f5708
Gecko: 8883360b1edb
Version: 31.0a1
Firmware Version: v1.2-device.cfg
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.