[B2G][Gallery][Edit Image] Saving edits for a large image causes the gallery app to OOM

NEW
Unassigned

Status

Firefox OS
Gaia::Gallery
3 years ago
3 years ago

People

(Reporter: Marty, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(b2g-v2.0 unaffected, b2g-v2.1 unaffected)

Details

(Whiteboard: [273MB-Flame-Support], URL)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
Created attachment 8460568 [details]
logcat-LMK.txt

Description:
If the user edits a large image, the Gallery app will OOM when the user attempts to save the edits.  For this to happen, the image must be roughly >800kb (most pictures taken by the camera are at least this size).  If the image is smaller (around 300kb), the image will be saved properly.


Repro Steps:
1) Update a Flame to 20140722040212
2) Take a picture with the camera.
3) Find that picture in the gallery, and select 'Edit'
4) Increase the brightness of the image, and press 'Done'


Actual:
The image is saved with the edits made by the user.

Expected:
The Gallery app experiences an OOM by an LMK


Environmental Variables:
Device: Flame Master (273mb)
Build ID: 20140722040212
Gaia: e423c3be8d19c9a8a5ae2571f499c36dc6b0df89
Gecko: 6f702709fab6
Version: 34.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0


Keywords: LMK, Gallery, Edit, OOM

Repro frequency: 9/10
See attached: logcat, video (URL).

------------------------------------------------------------

This issue DOES occur on Flame 2.0 273mb.
The Gallery app experiences an LMK when saving edits to a large image.

Environmental Variables:
Device: Flame Master (273mb)
Build ID: 20140722000200
Gaia: b9d19011123487009c80d1200937652d58c434a0
Gecko: 00f4b3a7046f
Version: 32.0 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0

------------------------------------------------------------

This issue does NOT occur on Flame 2.1 512mb, or Flame 2.0 512mb.
(Reporter)

Updated

3 years ago
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
QAWanted for branch checks for 1.4 to determine if this is a regression.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
Keywords: qawanted
This issue is no longer occurring on 319MB memory Flame 2.1 and Flame 2.0. I can now edit an ~800KB image, apply all the different effects on it and save it without the app crashing.

Under the same 2.1 build in 273MB mem the issue is 100% reproducible.

Tested on:
Device: Flame
Build ID: 20140730141509
Gaia: b67ddd7d40b52e65199478b8d6631c2c28fdf41d
Gecko: 005424a764da
Version: 34.0a1 (Master)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Device: Flame
Build ID: 20140731090608
Gaia: cbf288dafe5350d775a4b597dbce880fa5f3e3a2
Gecko: 0a5d3dadf49d
Version: 32.0 (2.0)
Firmware Version: v122
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.0: affected → unaffected
status-b2g-v2.1: affected → unaffected
Flags: needinfo?(jmitchell)
Keywords: qawanted
issue no longer repros in 319 mem, leaving open for perf investigation
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(jmitchell)

Updated

3 years ago
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
You need to log in before you can comment on or make changes to this bug.