Closed Bug 1614186 Opened 6 years ago Closed 6 years ago

Allow use of native Application Chooser to be pref controlled

Categories

(SeaMonkey :: Download & File Handling, enhancement)

enhancement
Not set
normal

Tracking

(seamonkey2.49esr wontfix, seamonkey2.53 fixed, seamonkey2.57esr fixed)

RESOLVED FIXED
seamonkey 2.72
Tracking Status
seamonkey2.49esr --- wontfix
seamonkey2.53 --- fixed
seamonkey2.57esr --- fixed

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

Details

(Whiteboard: SM2.53.2)

Attachments

(2 files, 1 obsolete file)

At the moment nsHelperAppDlg forces you to use the native application chooser if it is there, we should make it pref controlled with the default to be true.

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: forced to use native app chooser
Testing completed (on m-c, etc.): 2.53.x
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: none

Attachment #9125326 - Flags: review?(frgrahl)
Attachment #9125326 - Flags: approval-comm-release?
Attachment #9125326 - Flags: approval-comm-esr60?

[Approval Request Comment]
Regression caused by (bug #): n/a
User impact if declined: change has to be made in about:config
Testing completed (on m-c, etc.): 2.53.2
Risk to taking this patch (and alternatives if risky): none
String changes made by this patch: label and accesskey for pref pane and associated help

Attachment #9125500 - Flags: review?(frgrahl)
Attachment #9125500 - Flags: approval-comm-release?
Attachment #9125500 - Flags: approval-comm-esr60?

Switch to use XP_LINUX instead as suggested

Attachment #9125500 - Attachment is obsolete: true
Attachment #9125500 - Flags: review?(frgrahl)
Attachment #9125500 - Flags: approval-comm-release?
Attachment #9125500 - Flags: approval-comm-esr60?
Attachment #9127064 - Flags: review?(frgrahl)
Attachment #9127064 - Flags: approval-comm-release?
Attachment #9127064 - Flags: approval-comm-esr60?
Comment on attachment 9125326 [details] [diff] [review] pref control app chooser [Triage Comment] r/a=frg over IRC
Attachment #9125326 - Flags: review?(frgrahl)
Attachment #9125326 - Flags: review+
Attachment #9125326 - Flags: approval-comm-release?
Attachment #9125326 - Flags: approval-comm-release+
Attachment #9125326 - Flags: approval-comm-esr60?
Attachment #9125326 - Flags: approval-comm-esr60+
Comment on attachment 9127064 [details] [diff] [review] Expose pref and use in pref-applications v1.1 [Triage Comment] r/a=frg over IRC
Attachment #9127064 - Flags: review?(frgrahl)
Attachment #9127064 - Flags: review+
Attachment #9127064 - Flags: approval-comm-release?
Attachment #9127064 - Flags: approval-comm-release+
Attachment #9127064 - Flags: approval-comm-esr60?
Attachment #9127064 - Flags: approval-comm-esr60+

Pushed by iann_cvs@blueyonder.co.uk:
https://hg.mozilla.org/comm-central/rev/5eae61097a7e
Allow use of native Application Chooser to be pref controlled - expose pref part. r=frg DONTBUILD

Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED

Comm part only pushed to comm-central:
https://hg.mozilla.org/comm-central/rev/5eae61097a7e9ede09c2151df88eeaf6edb77ae7
Everything pushed to 2.53 and 2.57 SM repos.

Target Milestone: --- → seamonkey 2.72
Whiteboard: SM2.53.2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: