Closed Bug 1750466 Opened 2 years ago Closed 2 years ago

Unwanted mailto: handlers added to Thunderbird by default

Categories

(Thunderbird :: Upstream Synchronization, defect)

defect

Tracking

(thunderbird_esr91 unaffected)

RESOLVED FIXED
98 Branch
Tracking Status
thunderbird_esr91 --- unaffected

People

(Reporter: darktrojan, Assigned: darktrojan)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1733497 moved some regional settings for Firefox into uriloader/exthandler/HandlerList.jsm. These include mailto: handlers which are inappropriate in Thunderbird (clearly) and also cause any test which opens the Preferences tab to fail by attempting to load the GMail favicon.

I'm going to make a temporary fix for this so that we can get our tests back again. It may turn out to be a long-term fix if we determine that we aren't ever going to use data from HandlerList.jsm.

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f2a7875ccab9
Prevent the automatic addition of mailto: handlers. rs=bustage-fix

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

Oh, I broke some XPCShell tests instead … I probably should've expected that. Still, a net positive.

I'm not sure what a good way forward here is. I feel like HandlerList.jsm should belong in browser, with a mail version that has no handlers. But that may not be practical. Mike?

Status: RESOLVED → REOPENED
Flags: needinfo?(mozilla)
Resolution: FIXED → ---

That's an interesting idea, I just need to figure out where the file should go.

Or we can #ifdef THUNDERBIRD in the handler list...

Flags: needinfo?(mozilla)
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/integration/autoland/rev/fcc52abbaa35
Stop adding default mailto: handlers in Thunderbird. r=mkaply
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Backout by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/1edd582106a0
Backed out changeset f2a7875ccab9 because it is no longer needed.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: