Closed
Bug 632170
Opened 13 years ago
Closed 13 years ago
Downloaded files should be saved in a public place
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(fennec2.0b5+)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
fennec | 2.0b5+ | --- |
People
(Reporter: wesj, Assigned: wesj)
Details
(Keywords: regression)
Attachments
(2 files, 2 obsolete files)
3.09 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
2.00 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
Bug 622684 changed the method used for finding the downloads directory to use new APIs available in SDK 8 and greater. We're now calling getExternalFilesDir(Environment.DIRECTORY_DOWNLOADS) which returns something like: /Android/data/<package_name>/files/Download instead we can use the public downloads directory, recommended for "shared" files by http://developer.android.com/guide/topics/data/data-storage.html#filesExternal by using: getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS) which may even make things like images show up in the gallery again?.
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #510384 -
Flags: review?(blassey.bugs)
Comment 2•13 years ago
|
||
is it possible to continue to download our updates to the private location?
Assignee | ||
Comment 3•13 years ago
|
||
Yeah. We'd probably need a separate environmental variable, and then an update to mobile browser at: http://mxr.mozilla.org/mobile-browser/source/components/DirectoryProvider.js#77
tracking-fennec: --- → ?
Updated•13 years ago
|
tracking-fennec: ? → 2.0b5+
Updated•13 years ago
|
Assignee: nobody → wjohnston
Assignee | ||
Comment 4•13 years ago
|
||
Attachment #510384 -
Attachment is obsolete: true
Attachment #510708 -
Flags: review?(blassey.bugs)
Attachment #510384 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 5•13 years ago
|
||
Assignee | ||
Comment 6•13 years ago
|
||
Comment on attachment 510708 [details] [diff] [review] Patch v2 Removing review request till I've had a chance to test these. Sorry :(
Attachment #510708 -
Flags: review?(blassey.bugs)
Assignee | ||
Comment 7•13 years ago
|
||
Returns the directory given by the environmental variable "UPDATES_DIRECTORY" if it exists, otherwise uses the downloads directory. Note, updates are actually stored in this path + "/updates/".
Attachment #510710 -
Attachment is obsolete: true
Attachment #510838 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 8•13 years ago
|
||
Comment on attachment 510708 [details] [diff] [review] Patch v2 These return the correct directory for me on desktop and device. Not sure how you test actual updates...
Attachment #510708 -
Flags: review?(blassey.bugs)
Updated•13 years ago
|
Keywords: regression
Comment 9•13 years ago
|
||
Comment on attachment 510838 [details] [diff] [review] Mobile-browser v1.1 Do we care if the localFile exists or not? We could check that too. r+, but think about the check - we might not need it if we are being given the value in Java.
Attachment #510838 -
Flags: review?(mark.finkle) → review+
Updated•13 years ago
|
Attachment #510708 -
Flags: review?(blassey.bugs) → review+
Assignee | ||
Comment 10•13 years ago
|
||
pushed: mc: http://hg.mozilla.org/mozilla-central/rev/8728536ebec3 mb: http://hg.mozilla.org/mobile-browser/rev/da356a8d0644
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 11•13 years ago
|
||
VERIFIED FIXED on: Build Id: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b12pre) Gecko/20110210 Firefox/4.0b12pre Fennec /4.0b5pre Device: Motorola Droid 2 (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•