[Buri][REG][Gallery]The MS always crash when save picture edited

RESOLVED FIXED

Status

Firefox OS
Gaia::Gallery
P1
normal
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: sync-1, Assigned: dmarcos)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

log
2.27 MB, text/plain
Details
5.40 MB, application/octet-stream
Details
log
2.27 MB, text/plain
Details
223 bytes, text/html
johnhu
: review+
Details
(Reporter)

Description

4 years ago
Firefox os  v1.1
 Mozilla build ID:20130916041201
 
 Created an attachment (id=534193)
 log
 
 DEFECT DESCRIPTION:
  The MS always crash when save picture edited
 
 REPRODUCING PROCEDURES:
  1.Launch Gallery app
  2.select one picture to view
  3.Edit the pic
  4.Change its Effects,press finish button->MS crash->ko
  
 EXPECTED BEHAVIOUR:
  ko:the edited picture should be saved
 
 ASSOCIATE SPECIFICATION:
 
 TEST PLAN REFERENCE:
 
 TOOLS AND PLATFORMS USED:
 
 USER IMPACT:
  Mid
 
 REPRODUCING RATE:
  3/5
 
 For FT PR, Please list reference mobile's behavior:

Updated

4 years ago
blocking-b2g: --- → leo?

Comment 1

4 years ago
Hi I check the behavior  when loading
pics from sdcard, you are edit a pic and save it, the gallery easily crash.
(Reporter)

Comment 2

4 years ago
Clone from brother
(Reporter)

Comment 3

4 years ago
Created attachment 815768 [details]
log
(Reporter)

Comment 4

4 years ago
Clone from brother
(Reporter)

Comment 5

4 years ago
Created attachment 815769 [details]
video
(Reporter)

Comment 6

4 years ago
Clone from brother
(Reporter)

Comment 7

4 years ago
Created attachment 815770 [details]
log

Comment 8

4 years ago
When loading pics from sdcard, it consume some memory, and the same time the user edit and save pic also consume some memory. This time gallery easily killed by itself. Please help check and fix it. Thanks.

Updated

4 years ago
Flags: needinfo?(dmarcos)
Flags: needinfo?(dflanagan)
Ray, are you saying that this bug only occurs if you try to use the edit feature while the gallery is scanning the sdcard and processing images?

Does it happen when all the images on the sdcard are from the camera, or only when the sdcard has external images on it? 

The gallery's performance is much better when it works with images from the Camera and when the Camera includes an EXIF preview images that is at least as large as the screen size.

I see that this is marked leo? Are there still any 1.1 builds that are accepting patches?  I thought we were done with leo bugs...

In any case, if we need to patch this for 1.1, I'd say the simplest thing to do is disable the edit button while scanning is happening. That is better than crashing.

Diego: Do you want to take this one and do a simple "disable editing while scanning" patch?
Flags: needinfo?(dflanagan)

Comment 10

4 years ago
(In reply to David Flanagan [:djf] from comment #9)
> Ray, are you saying that this bug only occurs if you try to use the edit
> feature while the gallery is scanning the sdcard and processing images?

Yes. 

> Does it happen when all the images on the sdcard are from the camera, or
> only when the sdcard has external images on it? 
> 

Both kinds of circumstances will occur. And external images reproduce easily.

> The gallery's performance is much better when it works with images from the
> Camera and when the Camera includes an EXIF preview images that is at least
> as large as the screen size.
> 
> I see that this is marked leo? Are there still any 1.1 builds that are
> accepting patches?  I thought we were done with leo bugs...

Yeah, we could add patches by upgrade packs.

> 
> In any case, if we need to patch this for 1.1, I'd say the simplest thing to
> do is disable the edit button while scanning is happening. That is better
> than crashing.
> 
> Diego: Do you want to take this one and do a simple "disable editing while
> scanning" patch?
(Assignee)

Comment 11

4 years ago
I can have a look into this
(Assignee)

Updated

4 years ago
Assignee: nobody → dmarcos
Flags: needinfo?(dmarcos)
Hi David and Diego,

Does bug 915001 releate to this bug?
They all have some problems when saving the edited image.
(Assignee)

Comment 13

4 years ago
I made a patch with the solution that :djf suggested: we disable the edit button while mediadb scans pictures from storage. This is a temporary solution to prevent the gallery to crash.
(Assignee)

Comment 14

4 years ago
Created attachment 817548 [details]
pullRequest.html
(Assignee)

Updated

4 years ago
Attachment #817548 - Flags: review?(dflanagan)

Comment 15

4 years ago
(In reply to Diego Marcos from comment #14)
> Created attachment 817548 [details]
> pullRequest.html

Hi Diego,

I add your patch and test,it works. But I can also edit pics when scaning. And the step like follows: 
1.Launch Gallery app
2.click a pic  to preview.
3.Slide view on previous picture or the next picture.
4.Then click edit button quickly, user also can edit pics and this issue can happen.
(Assignee)

Comment 16

4 years ago
Thanks. Good catch! I updated the pull request to deal with the cases when the user pans to the previous or next image
Comment on attachment 817548 [details]
pullRequest.html

John,

Would you take this review for me?
Attachment #817548 - Flags: review?(dflanagan) → review?(johu)
Sure. I will do it today.
Comment on attachment 817548 [details]
pullRequest.html

Thanks for this patch. It looks good.
Attachment #817548 - Flags: review?(johu) → review+
(Assignee)

Comment 20

4 years ago
Merged on master:

https://github.com/mozilla-b2g/gaia/commit/6b0a8108eac92ab3b5e73946c57f750fe8744356
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Mass-modify - removal of no longer relevant blocking flags.

Updated

3 years ago
blocking-b2g: leo? → ---
You need to log in before you can comment on or make changes to this bug.