Download large images ( > 5MB) to disk when setting wallpaper from context menu

NEW
Unassigned

Status

()

Firefox for Android
General
P5
normal
5 years ago
2 years ago

People

(Reporter: sriramr, Unassigned, Mentored)

Tracking

Trunk
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [lang=java] )

(Reporter)

Description

5 years ago
This is a follow up for bug 715736. 

https://bugzilla.mozilla.org/show_bug.cgi?id=715736#c44

Currently the wallpaper is set by converting the image into a ByteArray for scaling purposes. Converting large images into a ByteArray can cause an OutOfMemoryException and thus result in failure of setting wallpaper. 

Thus, by downloading large images ( > 5MB) to disk we can sample the image and scale it down without running out of memory.
(Reporter)

Updated

5 years ago
Whiteboard: [lang=java]
(Reporter)

Updated

5 years ago
Hardware: x86_64 → ARM

Updated

5 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 1

5 years ago
Hi my partner and I are interested in fixing this bug. Can we get assigned to it? Is there any chance we could get a mentor for this, since this would be our first bug fix?
(In reply to Erica Hansen from comment #1)
> Hi my partner and I are interested in fixing this bug. Can we get assigned
> to it? Is there any chance we could get a mentor for this, since this would
> be our first bug fix?

Yup, I can mentor you no problem.
Whiteboard: [lang=java] → [lang=java] [mentor=snorp@snorp.net]
The patch in bug 901824 would also handle this, but is frozen :(
Is this bug still open to be taken? It seems like there was someone already assigned to the problem, so if someone is already working on this I won't intrude. Otherwise, I would gladly work on this.

Comment 5

4 years ago
I haven't had much time to work on this. So feel free to take it up.
(In reply to Tim Stullich from comment #4)
> Is this bug still open to be taken? It seems like there was someone already
> assigned to the problem, so if someone is already working on this I won't
> intrude. Otherwise, I would gladly work on this.
Sounds good. If possible could I be added as the assignee for this bug?
Also, what progress has been made on fixing this issue? Erika, perhaps you might be able to tell me.
Assignee: nobody → timstullich
The way things are set up right now is that when a large image is being downloaded there is no visual feedback being given to the user that a download is being performed, which I think should be changed. The application simply sits there while the image is being downloaded, which gives the impression to the user that nothing is happening. What would be a good way to handle this problem?
Flags: needinfo?(ibarlow)

Updated

4 years ago
Status: NEW → ASSIGNED

Updated

4 years ago
Flags: needinfo?(ibarlow)
(Assignee)

Updated

4 years ago
Mentor: snorp@snorp.net
Whiteboard: [lang=java] [mentor=snorp@snorp.net] → [lang=java]
Assignee: timstullich → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.