Remove toolkit handler binding

RESOLVED FIXED in Firefox 67

Status

()

P3
normal
RESOLVED FIXED
2 months ago
24 days ago

People

(Reporter: ntim, Assigned: surkov)

Tracking

(Blocks: 2 bugs)

unspecified
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Reporter)

Description

2 months ago

https://searchfox.org/mozilla-central/source/toolkit/mozapps/handling/content/handler.xml

https://dxr.mozilla.org/mozilla-central/search?q=type+handler+path%3Atoolkit++path%3A.js+-path%3Aobj&redirect=false

The markup could just be built directly in the JS file. There are 4 different places, but it'd be possible to create a shared function for these 4 use cases that append the markup inside the richlistitem.

(Reporter)

Updated

2 months ago
Blocks: 1512432
(Reporter)

Comment 1

2 months ago
Posted image Handler dialog
Priority: -- → P3

There's also a patch in progress at Bug 1519486 that will import the base richlistitem custom element, and this would be a fairly easy extension of that (using [is] / extends) if we'd rather do it that way.

(Assignee)

Comment 4

25 days ago

how to test this?

Assignee: nobody → surkov.alexander

(In reply to alexander :surkov (:asurkov) from comment #4)

how to test this?

Go into about:preferences and set Applications -> mailto -> Always ask then load: data:text/html,<a href="mailto:someone@somewhere.com">mail</a> and click on the link.

Attachment #9047827 - Attachment is obsolete: true

Comment 8

25 days ago
Pushed by asurkov@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0c13462c5416
Remove toolkit handler binding, r=bgrins

Comment 9

24 days ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 24 days ago
status-firefox67: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.