1521306 - Remove toolkit handler binding

Status: RESOLVED FIXED

(Toolkit :: General, task, P3)

Description

[Tim Nguyen :ntim]

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.

Comment 1

[Tim Nguyen :ntim]

Attachment: Handler dialog

Comment 2

[Tim Nguyen :ntim]

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

Comment 3

[Brian Grinstead [:bgrins]]

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.

Comment 4

[alexander :surkov (:asurkov)]

how to test this?

Comment 5

[Brian Grinstead [:bgrins]]

(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:[e-mail]">mail</a> and click on the link.

Comment 6

[alexander :surkov (:asurkov)]

Attachment: Bug 1521306 - Remove toolkit handler binding, r=bgrins

Comment 7

[alexander :surkov (:asurkov)]

Attachment: Bug 1521306 - Remove toolkit handler binding, r=bgrins

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0c13462c5416
Remove toolkit handler binding, r=bgrins

Comment 9

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/0c13462c5416