826507 - Switching between camera to video recording crashes pretty consistently

Status: RESOLVED INVALID

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

Description

[Marcia Knous [:marcia]]

Attachment: Logcat filtered on camera during crash scenario

Similar to David's Bug 824435, but this is a crash and not a freeze. Bug 820139 was fixed, so maybe that fix didn't address the entire issue. Seen while running unagi using:

Gecko:
Gaia:

STR:
1. Start with the camera mode and take a picture.
2. Move to video recording mode and start a video and stop it.
3. After that I switch back and forth rapidly between camera mode and video mode.
4. Eventually I get a message that "Camera just crashed" or "just crashed"

Sometimes I notice that it happens more regular when I tap one of the buttons rapidly.

My crash report: https://crash-stats.mozilla.com/report/index/e72d94c3-542b-4335-a194-1baff2130103 - however it is unfortunately not useful.

Comment 1

[Joe Cheng [:jcheng] (please needinfo)]

Triage: BB-, tracking-b2g18+
seems only reproducible if you do the switch very quickly and intentionally trying to break it.

Comment 2

[David Flanagan [:djf]]

cc'ing Dale.

Feel free to steal this if you want it Dale.

Comment 3

[David Flanagan [:djf]]

I suppose there could be a timing issue here... while trying to initialize whatever camera hardware is being initialized, the user switches back and forth and triggers another call to the camera initialization code.  If that's the problem the easiest solution might be to disable the switch button until the switch is complete.

Comment 4

[Dale Harvey (:daleharvey)]

stoled

Comment 5

[Dale Harvey (:daleharvey)]

The switch button is only enabled once the previewstream is enabled, we had this bug before and the fix was tested quite extensively. I have tried reproducing it on both otoro and unagi and cant.

It seems like this must be a build issue, I have tried flashing several builds

Comment 6

[Dale Harvey (:daleharvey)]

mike this is a pretty vague bug, would it have been fixed by various other camera related fixes? I am unable to reproduce.

Comment 7

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

daleharvey, yeah, I believe we've addressed this with a number of fixen.

marcia, do you still see this at all?

Comment 8

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

marcia, do you still see this issue at all? If not, I think we can close it.

Comment 10

[Marcia Knous [:marcia]]

On V1 train there is a crash whenever you touch video recording, so I can retry this when that bug has been addressed.

Comment 11

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

(In reply to Marcia Knous [:marcia] from comment #10)
>
> On V1 train there is a crash whenever you touch video recording, so I can
> retry this when that bug has been addressed.

Yeah, that is (was?) bug 872170.  Thanks, Marcia!

Comment 12

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

Hi Sarah, do your extra tags mean you are still able to reproduce this specific issue?  (As opposed to the issue in bug 872170?)

Comment 13

[Sarah Parsons]

Hey Mike, I added those tags before I saw your last comment (comment 11) The issue I reproduced was bug 872170. Sorry about that!

Comment 14

[Diego Marcos [:dmarcos]]

Bugs cleanup. This is obsolete. After the camera refactor/redesig that landed in 1.4 this doesn't apply anymore.