Bug 1750042 Comment 11 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

### Beta/Release Uplift Approval Request
* **User impact if declined**: Local files don't open the way they should (create additional copies)
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: See comment 0. Make sure the filetype in question is set to "always ask" in the Firefox settings.

Alternatively, I believe you can also reproduce (without setting the policy prefs) by just opening a directory listing inside Firefox (e.g. opening `file:///C:/Users/<username>/Downloads/` and then clicking a link to a filetype that is set to "always ask")
* **List of other uplifts needed**: n/a
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Fairly straightforward patch that adds back functionality accidentally removed during a refactor, and adds an automated test so we don't regress it again. Plus we still have some beta runway.
* **String changes made/needed**: None
### Beta/Release Uplift Approval Request
* **User impact if declined**: Local files don't open the way they should (create additional copies)
* **Is this code covered by automated tests?**: Yes
* **Has the fix been verified in Nightly?**: No
* **Needs manual test from QE?**: Yes
* **If yes, steps to reproduce**: See comment 0. Make sure the filetype in question is set to "always ask" in the Firefox settings.

Alternatively, I believe you can also reproduce (without setting the policy prefs) by just opening a directory listing inside Firefox (e.g. opening `file:///C:/Users/<username>/Downloads/`) and then clicking a link to a filetype that is set to "always ask"
* **List of other uplifts needed**: n/a
* **Risk to taking this patch**: Low
* **Why is the change risky/not risky? (and alternatives if risky)**: Fairly straightforward patch that adds back functionality accidentally removed during a refactor, and adds an automated test so we don't regress it again. Plus we still have some beta runway.
* **String changes made/needed**: None

Back to Bug 1750042 Comment 11