Closed
Bug 1226120
Opened 9 years ago
Closed 9 years ago
Write regression test for bug 1040005 (handling of the "open with" list for alwaysAsk files when the default used to be something other than opening)
Categories
(Core Graveyard :: File Handling, defect)
Core Graveyard
File Handling
Tracking
(firefox45 affected, firefox46 fixed)
RESOLVED
FIXED
mozilla46
People
(Reporter: Gijs, Assigned: Gijs)
References
Details
Attachments
(1 file, 1 obsolete file)
See bug 1040005 for problem description. We should write a test, also because I expect we don't have an awful lot of tests of this functionality at the moment.
Assignee | ||
Comment 1•9 years ago
|
||
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8699517 [details] [diff] [review] add test for unknownContentType.xul and file types with a default handler but always ask set, Gah, meant to just hg export.
Attachment #8699517 -
Attachment is obsolete: true
Assignee | ||
Comment 3•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/28267/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/28267/
Attachment #8699523 -
Flags: review?(mconley)
Assignee | ||
Comment 4•9 years ago
|
||
I verified that this fails on beta and passes locally for me on OSX. Trypush: remote: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9d22bf01848d
Comment 5•9 years ago
|
||
Comment on attachment 8699523 [details] MozReview Request: Bug 1226120 - add test for unknownContentType.xul and file types with a default handler but always ask set, r?mconley https://reviewboard.mozilla.org/r/28267/#review25323 ::: uriloader/exthandler/tests/mochitest/browser_download_always_ask_preferred_app.js:94 (Diff revision 1) > + helperAppDialog.show(launcher, window, "foopy"); Indentation ::: uriloader/exthandler/tests/mochitest/browser_download_always_ask_preferred_app.js:96 (Diff revision 1) > + Cu.reportError(ex); Should we fail here? ::: uriloader/exthandler/tests/mochitest/browser_download_always_ask_preferred_app.js:99 (Diff revision 1) > + yield new Promise(resolve => { Could also use BrowserTestUtils.waitForEvent, no?
Attachment #8699523 -
Flags: review?(mconley) → review+
Comment 7•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a8e74b6b0511
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox46:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•