Closed Bug 600300 Opened 9 years ago Closed 9 years ago

Save helper app downloads to downloads directory on Android and Maemo

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set

Tracking

(fennec2.0b2+)

VERIFIED FIXED
Tracking Status
fennec 2.0b2+ ---

People

(Reporter: wesj, Assigned: wesj)

Details

Attachments

(2 files)

Android stock browser saves everything to downloads directory. We currently save downloaded files there, but files opened with helper apps are saved in the temp directory. We should match the android browser behavior.
Patch from Bug 596370.
Comment on attachment 479127 [details] [diff] [review]
Patch from Bug 596370

This patch should make Fennec work like the Android browser regarding file download locations and only downloading to the /sdcard
Attachment #479127 - Flags: review?(blassey.bugs)
Attachment #479127 - Flags: review?(blassey.bugs) → review+
Let's make sure we get this behavior in for b2
tracking-fennec: --- → 2.0b2+
Assignee: nobody → wjohnston
pushed:
http://hg.mozilla.org/mozilla-central/rev/122f096c90c2
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Flags: in-testsuite?
Flags: in-litmus?
Verified:
Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101018 Firefox/4.0b8pre Fennec/4.0b2pre

Note to tester (from wesj): 
app helper is defined as anything that automatically gets opened by an external application.  in fennec, that's everything except for downloads that you long click on and pick "Save Link" from the context menu.

ie. like a short tap on a pdf file as long as there is a pdf reader installed should show this.
Status: RESOLVED → VERIFIED
Flags: in-litmus? → in-litmus?(ayanshah62)
https://litmus.mozilla.org/show_test.cgi?id=13818
Flags: in-litmus?(ayanshah62) → in-litmus+
A followup patch was pushed:
http://hg.mozilla.org/mozilla-central/rev/1a55741b7b27

which started burning the Maemo trees. The MAEMO code was never getting compiled and now that it is, it's burning. Fix coming up.
Bustage fix
Attachment #492571 - Flags: review?(blassey.bugs)
Attachment #492571 - Flags: review?(blassey.bugs) → review+
Summary: Save helper app downloads to downloads directory on Android → Save helper app downloads to downloads directory on Android and Maemo
You need to log in before you can comment on or make changes to this bug.