Use Downloads.jsm functions to get download directories in the Safari Migrator

VERIFIED FIXED in Firefox 37

Status

()

VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: Paolo, Assigned: Paolo)

Tracking

Trunk
Firefox 37
Points:
1
Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 8537206 [details] [diff] [review]
Untested patch

This is part of bug 875648.

I've not added a wrapper for the outer Task, similar to the one for the synchronous function, because there is only one instance for now.
Attachment #8537206 - Flags: review?(mak77)
Flags: firefox-backlog+
(Assignee)

Updated

4 years ago
OS: Windows 7 → All
Hardware: x86_64 → All
Comment on attachment 8537206 [details] [diff] [review]
Untested patch

Review of attachment 8537206 [details] [diff] [review]:
-----------------------------------------------------------------

It looks good.
Unfortunately we don't have automated tests covering this, do you have the opportunity to test this manually?

Ideally, we could even write a mac-only xpcshell-test checking it, but it would require to also set the downloads folder (DownloadsPath) in the system plist, and we only have a plist reader, not a writer.
Attachment #8537206 - Flags: review?(mak77) → review+
(Assignee)

Comment 2

4 years ago
(In reply to Marco Bonardo [::mak] (needinfo? me) from comment #1)
> Unfortunately we don't have automated tests covering this, do you have the
> opportunity to test this manually?

I tested this right now, setting the preference to the Desktop, Downloads, and a custom folder.

There was an import missing but the rest worked just fine. Landed on fx-team:

https://hg.mozilla.org/integration/fx-team/rev/38471b0310c9
https://hg.mozilla.org/mozilla-central/rev/38471b0310c9
Assignee: nobody → paolo.mozmail
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37

Updated

4 years ago
Iteration: --- → 37.3
Flags: qe-verify?
(Assignee)

Comment 4

4 years ago
This can be tested by changing the default download location in Safari and importing preferences from Safari (<https://support.mozilla.org/en-US/kb/import-bookmarks-data-another-browser>).
Flags: qe-verify? → qe-verify+
QA Contact: catalin.varga
Can this be tested manually? If yes, can you please provide a more detailed scenario?
Flags: needinfo?(paolo.mozmail)
(Assignee)

Comment 6

4 years ago
(In reply to Catalin Varga [QA][:VarCat] from comment #5)
> Can this be tested manually? If yes, can you please provide a more detailed
> scenario?

Yes, firstly you should open Safari and change the download location from its preferences window.

There are three cases to test:
- Desktop folder
- Downloads folder
- Custom folder

For each of these, you should save the setting in Safari, then switch to Firefox and go through the wizard that allows you to import those settings:

https://support.mozilla.org/en-US/kb/import-bookmarks-data-another-browser

You should verify that the corresponding download location in the Firefox preferences has been updated correctly.

Hope this helps!
Flags: needinfo?(paolo.mozmail)
Thanks for the prompt feedback.

Verified the fix using the following environment:

FF 37 Developer Edition
Build Id:20150212004049
OS: Mac Os X 10.9.5, Win 7 x64

the corresponding download location in Firefox preferences is successfully updated after each import of data from Safari.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 8

4 years ago
(In reply to Catalin Varga [QA][:VarCat] from comment #7)
> OS: Mac Os X 10.9.5, Win 7 x64

Thanks for testing on Windows too!
You need to log in before you can comment on or make changes to this bug.