996361 - [Tarako][Camera] After taking very large, hi res picture (memory pressure), both edit mode and gallery fail to subsequently reload

Status: RESOLVED WORKSFORME

(Firefox OS Graveyard :: Gaia::Gallery, defect, P3)

Description

[John Hammink]

Attachment: dmesg.log

[Device]
  Tarako
---------------------------------------------
Gaia      0e7f21e61625b75a9149480cd5a259211549f020
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3t/rev/3b02811c314a
BuildID   20140414004001
Version   28.1
ro.build.version.incremental=215
ro.build.date=Mon Mar 31 10:11:50 CST 2014
---------------------------------------------
[STR]
1. Launch Camera.  Position "Color Test" picture on monitor at center of viewport, focusing on colorful area.  Take several pictures.  Try to edit some of the pictures.

---------------------------------------------
[Expected Result]
Picture can be edited, saved to gallery as edited and gallery app closed gracefully.

---------------------------------------------
[Actual Result]
Gallery mode hangs on edit preview.  Eventually gallery app crashes.  After restarting the gallery, gallery goes into loop and continually shows progress, but never seems to fully load.

Comment 1

[John Hammink]

Attachment: color_test1mb.png

Comment 2

[John Hammink]

Attachment: gallery_in_loading_loop.log

Comment 3

[John Hammink]

Comment on attachment 8406525 [details]
gallery_in_loading_loop.log

adb logcat of gallery stuck in loop during reload

Comment 4

[John Hammink]

Possibly related to: https://bugzilla.mozilla.org/show_bug.cgi?id=995148

Comment 5

[Jason Smith [:jsmith]]

Can we confirm we can't repro this on Buri 1.3?

Comment 6

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

It appears that this does not happen on buri given the STRs.  Perhaps with even a larger picture file?
Gaia      03340f9b00343512066ab0dceca1641ef9a5374c
Gecko     https://hg.mozilla.org/releases/mozilla-b2g28_v1_3/rev/0b0822ffdbf8
BuildID   20140414164003
Version   28.0
ro.build.version.incremental=324
ro.build.date=Thu Dec 19 14:04:55 CST 2013
Buri

OOM in John's Log:
<4>0[ 2149.188707] select 3341 (Camera), adj 10, size 7141, to kill
<4>0[ 2149.188773] send sigkill to 3341 (Camera), adj 10, size 7141

Comment 7

[Jason Smith [:jsmith]]

Can we get reduced STR for this bug explicitly calling out each step that was used?

Comment 8

[Pi Wei Cheng [:piwei] (inactive)]

(In reply to Jason Smith [:jsmith] from comment #7)
> Can we get reduced STR for this bug explicitly calling out each step that
> was used?

I was able to reproduce the gallery crash when saving an edited image, but not the subsequent behavior (gallery goes into loop and continually shows progress, but never seems to fully load). I've been testing specifically on editing images in Gallery today and I've not seen this subsequent behavior.

And I believe the gallery crash is being covered more clearly in bug 998095.

Tested on:
Device: Tarako 1.3T MOZ
BuildID: 20140424014003
Gaia: 706f474230432c8cb01e124aee2c055ec902aa1d
Gecko: 26ef3dded9ff
Version: 28.1
Firmware Version: sp6821a_gonk4.0_user.pac

Comment 9

[Jason Smith [:jsmith]]

Closing as wfm then per comment 8.