Open Bug 393502 Opened 17 years ago Updated 2 years ago

new protocol handling dialog should add preferred handler to set of possible handlers

Categories

(Firefox :: File Handling, defect)

defect

Tracking

()

People

(Reporter: myk, Unassigned)

Details

When the new protocol handling dialog sets the preferred handler for a given type, it should also add that handler to the set of possible handlers so that the datasource reflects the data as the API says it will be.

This just needs a one-liner added after http://mxr.mozilla.org/mozilla/source/toolkit/mozapps/handling/content/dialog.js#194

Note: bug 393500 and the way the dialog happens to use the handler info object means that not doing this won't cause a problem, but others might come along and copy this code later, not realizing about the potential issue, and then use the handler info object in a way that does expose this problem.

So it's worth fixing.
Severity: minor → S4
You need to log in before you can comment on or make changes to this bug.