Closed
Bug 915876
Opened 11 years ago
Closed 11 years ago
[B2G] [Buri] [1.2] [Gallery] Attempts to edit pictures do not get saved
Categories
(Firefox OS Graveyard :: Gaia::Gallery, defect)
Tracking
(blocking-b2g:koi+, firefox27 affected, b2g-v1.2 affected)
People
(Reporter: ckreinbring, Assigned: sotaro, NeedInfo)
References
Details
(Keywords: regression, Whiteboard: [burirun1][fromAutomation][xfail], burirun3)
Attachments
(3 files)
Description:
Making changes to a picture and attempting to save does not save the changed picture.
Repro Steps:
1) Update Buri to Build ID: 20130912040201
2) Launch the Gallery app.
3) Select a picture then tap the Edit icon.
4) Make a change to the picture then tap the Done icon.
5) Observe the reaction on the screen.
Actual:
Done icon fades and becomes unselectable, but the changed picture is not saved.
Expected:
The user is returned to the grid view with the changed picture added to the upper left corner.
Environmental Variables
Occurs on the Buri 1.2 mozilla RIL
Build ID: 20130912040201
Gecko: http://hg.mozilla.org/mozilla-central/rev/a98569f21abe
Gaia: 9ffd2899eb91388f7fc1ce6f7a895a6f5f922c05
Platform Version: 26.0a1
Notes:
Repro frequency: 85%
See attached logcat logs
Reporter | ||
Updated•11 years ago
|
Updated•11 years ago
|
QA Contact: laliaga
Updated•11 years ago
|
Comment 1•11 years ago
|
||
Tested as far back as 6/21 (our earliest) on Buri. Unable to save edited photos.
- Broken 1.2 -
Environmental Variables
Build ID: 20130621031231
Gecko: http://hg.mozilla.org/mozilla-central/rev/7ba8c86f1a56
Gaia: e2f19420fa6a26c4287588701efaec09a750dba1
Platform Version: 24.0a1
Tested on a Leo device 1.1 and it's working. Able to save edited photos.
- Working 1.1 -
Environmental Variables
Build ID: 20130909041201
Gecko: http://hg.mozilla.org/releases/mozilla-b2g18/rev/796d0adce82b
Gaia: 6324c3b047aabee83ec7419803224e8edf07db7c
Platform Version: 18.1
Keywords: regressionwindow-wanted
Comment 2•11 years ago
|
||
Adding needs info for Lucas - we need the last time this worked in a build, and the first build it was broken in.
Flags: needinfo?(laliaga)
Comment 3•11 years ago
|
||
(In reply to Marcia Knous [:marcia] from comment #2)
> Adding needs info for Lucas - we need the last time this worked in a build,
> and the first build it was broken in.
I think what Lucas is saying here is that he's gone far enough back to confirm that the regression is occurring all the way back in June on central. Do we even have builds to go farther back than this?
FWIW - I think regression windows are really only going to get their value if it's within a 3 - 4 month timeframe. Any farther than that will introduce too many changes, so we'll likely need a forward fix as a result.
Updated•11 years ago
|
Whiteboard: burirun1
Updated•11 years ago
|
Flags: needinfo?(laliaga) → needinfo?(mozillamarcia.knous)
Comment 4•11 years ago
|
||
I was asked to retest this on 1.2 for triage team. On Buri using:
Gaia 88e73da95f1c550f2fb0572480a40c989d37c997
SourceStamp 46b216260c1d
BuildID 20130920004004
Version 26.0a2
I am unable to edit a picture at all, because whenever I made the change and tap the "check" button, nothing happens. See attached logcat there are numerous "unknown deliver target" messages. This affects the latest Mozilla Central Build as well.
Flags: needinfo?(mozillamarcia.knous)
Updated•11 years ago
|
status-firefox27:
--- → affected
Assignee | ||
Comment 5•11 years ago
|
||
From attachment 808055 [details], it seems to fail to create GLContext because of out of pmem
------------------------------------------------
E/msm7627a.gralloc(141): gralloc failed err=Out of memory
W/GraphicBufferAllocator(141): alloc(1200, 1600, 1, 00000300, ...) failed -12 (Out of memory)
E/GeckoConsole(495): [JavaScript Warning: "Error: WebGL: Can't get a usable WebGL context" {file: "app://gallery.gaiamobile.org/js/ImageEditor.js" line: 1151}]
E/GeckoConsole(495): [JavaScript Error: "TypeError: gl is null" {file: "app://gallery.gaiamobile.org/js/ImageEditor.js" line: 1154}]
Assignee | ||
Comment 6•11 years ago
|
||
Current v1.2 and master's WebGL does not support memory fallback during out of pmem. Then If this happens WebGL context creation just fail. There seems two choices in Bug 905882 comment 12 to handle this condition. Need to make clear and decide choose which one.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → sotaro.ikeda.g
Comment 7•11 years ago
|
||
Based on comment #4 with the retest results saying edit is broken and last triage discussion, making this a koi+
Updated•11 years ago
|
blocking-b2g: koi? → koi+
Assignee | ||
Comment 8•11 years ago
|
||
Both attachment 808055 [details] and attachment 804009 [details] have following log. From the log, it became clear that "fallback during out of pmem" were disabled.
> E/memalloc(154): /dev/pmem: No more pmem available
> E/msm7627a.gralloc(154): gralloc failed err=Out of memory
> W/GraphicBufferAllocator(154): alloc(1000, 634, 1, 00000300, ...) failed -12 (Out of memory)
Assignee | ||
Comment 9•11 years ago
|
||
I locally created the following ROM. By using the ROM, edited picture save seems always succeeded on my master hamachi.
- Enable out of pmem fallback by reverting Bug905784.
- Disable HwComposer to prevent rendering problem.
From this, if b2g handles 'out of pmem' correctly, this problem seems to be solved.
Assignee | ||
Comment 10•11 years ago
|
||
Right now, Bug 905882 seems most graceful fallback.
Comment 11•11 years ago
|
||
Following the STRs in Comment 0 on Mozilla Build Master/m-c the actual results were reproduced - Done icon fades and becomes unselectable, but the changed picture is not saved. This is the logcat for the below
Environmental Variables
Gaia 959ac2f692d85072ffdc3d16a041b5bf4735ae59
Gecko aa986b6ce882
BuildID 20131010040202
Version 27.0a1
Updated•11 years ago
|
Whiteboard: burirun1 → [burirun1][fromAutomation]
Updated•11 years ago
|
Whiteboard: [burirun1][fromAutomation] → [burirun1][fromAutomation][xfail]
Comment 12•11 years ago
|
||
Hema,
What is the ETA for this koi+ issue to be fixed?
Are we dependent on 905882 for 915876 to be resolved?
Flags: needinfo?(hkoka)
Comment 13•11 years ago
|
||
Waiting on Bug 905882 to land, to help here.
Updated•11 years ago
|
Flags: needinfo?(hkoka)
Comment 15•11 years ago
|
||
This issue reproduces on Buri 1.2 Build ID: 20131101004000
Gaia e717aec947571f5daf923c040a82f9f0719bb526
SourceStamp 54de309e18a9
BuildID 20131101004000
Version 26.0
This issue also reproduces on Buri 1.3 Build ID: 20131101040203
Gaia ccdf357ea150fc7d8b8a4b74c7adf31e7a57e465
SourceStamp abe6790a5dd8
BuildID 20131101040203
Version 28.0a1
Keywords: qawanted
Updated•11 years ago
|
Whiteboard: [burirun1][fromAutomation][xfail] → [burirun1][fromAutomation][xfail], burirun3
Updated•11 years ago
|
Target Milestone: --- → 1.3 Sprint 4 - 11/8
Comment 16•11 years ago
|
||
Past Target Milestone and still repros on 1.2
Environmental Variables:
Device: Buri 1.2 mozRIL
BuildID: 20131111004004
Gaia: 670b2c8329bca6f142939185be71274166d82bb8
Gecko: 1ac147e4e2f0
Version: 26.0
Firmware Version: 20131104
status-b2g-v1.2:
--- → affected
Fixed on OEM version.
Gaia 2ef9bc3c7a6de228b63e6ba3613eb0c0dd639c59
BuildID 20131105151351
Version 26.0a2
1.2 Buri
Will retest on our own gecko/gaia on top of OEM build after finishing further OEM testing
Fixed on
Gaia: 377eb71506d8790d3c3280d82ac007ff4525b7e0
Gecko: c15f5d7e0d7ebff5f8edef0745f144cd1552e34c
BuildID 20131112123250
Version 28.0a1
Base Build : US_V1.2_20131111.cfg
Buri
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
Comment 21•11 years ago
|
||
The issue is not fixed on the hamachi yet. Tried on:
Gaia: 67866e82379e751646aa5f0fd6a7a4268e9529c1
Gecko: http://hg.mozilla.org/mozilla-central/rev/597287004ff5
BuildID 20131120062258
Version 28.0a1
Edited picture doesn't get saved and I get the following log messages:
E/msm7627a.gralloc( 140): gralloc failed err=Out of memory
W/GraphicBufferAllocator( 140): alloc(1024, 1024, 1, 00000300, ...) failed -12 (Out of memory)
E/GeckoConsole( 732): [JavaScript Warning: "Error: WebGL: Can't get a usable WebGL context" {file: "app://gallery.gaiamobile.org/js/ImageEditor.js" line: 1306}]
Helix works as expected on:
Gaia: c26480b22ce28c812c347290dd4bad090d83db6f
Gecko: http://hg.mozilla.org/mozilla-central/rev/597287004ff5
BuildID 20131120062258
Version 28.0a1
Updated•11 years ago
|
Flags: needinfo?(ikeda.sohtaroh)
Comment 22•11 years ago
|
||
(In reply to Diego Marcos from comment #21)
> The issue is not fixed on the hamachi yet. Tried on:
>
> Gaia: 67866e82379e751646aa5f0fd6a7a4268e9529c1
> Gecko: http://hg.mozilla.org/mozilla-central/rev/597287004ff5
> BuildID 20131120062258
> Version 28.0a1
>
> Edited picture doesn't get saved and I get the following log messages:
>
> E/msm7627a.gralloc( 140): gralloc failed err=Out of memory
> W/GraphicBufferAllocator( 140): alloc(1024, 1024, 1, 00000300, ...) failed
> -12 (Out of memory)
> E/GeckoConsole( 732): [JavaScript Warning: "Error: WebGL: Can't get a
> usable WebGL context" {file:
> "app://gallery.gaiamobile.org/js/ImageEditor.js" line: 1306}]
>
> Helix works as expected on:
>
> Gaia: c26480b22ce28c812c347290dd4bad090d83db6f
> Gecko: http://hg.mozilla.org/mozilla-central/rev/597287004ff5
> BuildID 20131120062258
> Version 28.0a1
What base image are you using?
Comment 23•11 years ago
|
||
How can I tell the base image?
Diego, I think you're still on 1.1 if you haven't flashed the device with the windows flashing tool.
You will want to flash with the v1.2_20131115.cfg build.
To note :
adb shell cat /system/build.prop | grep "ro.build.date"
The v1.2_20131115 build will have :
ro.build.date=Thu Nov 14 10:58:33 CST 2013
ro.build.date.utc=1384397913
Diego checked with me and we verified that he was on an older base build.
Assignee | ||
Comment 26•11 years ago
|
||
(In reply to Diego Marcos from comment #21)
> The issue is not fixed on the hamachi yet. Tried on:
>
> Gaia: 67866e82379e751646aa5f0fd6a7a4268e9529c1
> Gecko: http://hg.mozilla.org/mozilla-central/rev/597287004ff5
> BuildID 20131120062258
> Version 28.0a1
>
> Edited picture doesn't get saved and I get the following log messages:
>
> E/msm7627a.gralloc( 140): gralloc failed err=Out of memory
> W/GraphicBufferAllocator( 140): alloc(1024, 1024, 1, 00000300, ...) failed
> -12 (Out of memory)
> E/GeckoConsole( 732): [JavaScript Warning: "Error: WebGL: Can't get a
> usable WebGL context" {file:
> "app://gallery.gaiamobile.org/js/ImageEditor.js" line: 1306}]
>
This log says pmem fallback did not work on the ROM. If the fallback happens the following log should be there.
> E/memalloc(137): /dev/pmem: No more pmem available
> W/memalloc(137): Falling back to ashmem
Comment 27•11 years ago
|
||
I flashed my hamachi with the latest OEM build.
I still cannot save edited images on gallery. Versions installed:
Gaia 0659f16b9790b1cf9eba4d80743fcc774d2ffe3a
Gecko http://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/af2c7ebb5967
BuildID 20131205004003
Version 26.0
ro.build.version.incremental=eng.archermind.20131114.105818
Steps to reproduce:
1. Open Camera
2. Take Picture
3. Go to Gallery
4. Edit and save the picture multiple. Most of the times the gallery is killed at the 2nd attempt.
Status: VERIFIED → REOPENED
Resolution: WORKSFORME → ---
Comment 28•11 years ago
|
||
(In reply to Diego Marcos from comment #27)
> I flashed my hamachi with the latest OEM build.
>
> I still cannot save edited images on gallery. Versions installed:
>
> Gaia 0659f16b9790b1cf9eba4d80743fcc774d2ffe3a
> Gecko http://hg.mozilla.org/releases/mozilla-b2g26_v1_2/rev/af2c7ebb5967
> BuildID 20131205004003
> Version 26.0
> ro.build.version.incremental=eng.archermind.20131114.105818
>
> Steps to reproduce:
>
> 1. Open Camera
> 2. Take Picture
> 3. Go to Gallery
> 4. Edit and save the picture multiple. Most of the times the gallery is
> killed at the 2nd attempt.
There's other bugs open for tracking this. See bug 939962, bug 941160, and bug 945774. They are all focusing on the Skia GL specific issue involved here.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•