945185 - Show only first N images in image uploader for low-end devices.

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia, defect)

Description

[Thinker Li [:sinker]]

Story:
Image uploader would show all images before uploading them to flickr or other services.  It causes OOM if the number of images is too big.

Solution:
Show only limited number of images at image uploader, but upload all images to the service.

Comment 1

[Andreas Gal :gal]

Can't block on features days before 1.3 FC. Also, this is not the right fix. We should downscale automatically and make sure images don't take too much memory instead of artificially limiting image count.

Comment 2

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

triage: similar to bug 854795, close this as dup

Comment 3

[David Flanagan [:djf]]

Reopening this because it really isn't a dupe of 854795.  That bug is speculative and is not guaranteed to be fixed in 1.4. Even if it is, it won't automatically fix problems like this one. Fixes will still require Gaia side changes to every affected app.

Thinker: is the image uploader actually part of the build and shipping on devices? Is this the bluetooth transfer feature?  If multiple image selection is causing OOMs, then we should probably have a crash keyword on this bug and we should fix it.

One fix would be to modify gallery so that it doesn't allow more than a few images to be selected at a time. But I don't really like that.

Another fix is as proposed: don't display all of them.

Another fix would be to modify gallery so that the share activity includes thumbnails for each of the shared images.  That would allow receiving apps to display thumbnails without ever decoding the full-size image.

Comment 4

[Thinker Li [:sinker]]

This bug would be fixed by 854795.  But, like David had said, the schedule of that bug is not clear now.  So, I change my mind, I agree with David to keep this bug opened for a short term solution.

Comment 5

[Andrew Overholt [:overholt]]

Triage decided this still doesn't seem like a blocker for 1.3.  It may be for 1.4, though, so I'm moving the nom to there.

Comment 6

[Preeti Raghunath(:Preeti)]

Moving to 1.3T? for review

Comment 7

[Tim Guan-tin Chien [:timdream] (please needinfo)]

There isn't an image uploader in Gaia. If we are talking about the experimental Twitter Share app, it was removed in bug 959447.

Comment 8

[Tim Guan-tin Chien [:timdream] (please needinfo)]

OK, I was wrong. There IS a Image Uploader app in test_apps.

https://github.com/mozilla-b2g/gaia/commits/master/test_apps/image-uploader

This remain a valid bug but not a feature we are shipping.

Comment 9

[BMO Automation]

Firefox OS is not being worked on