Closed Bug 1521306 Opened 2 years ago Closed 2 years ago

Remove toolkit handler binding

Categories

(Toolkit :: General, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: ntim, Assigned: surkov)

References

Details

Attachments

(2 files, 1 obsolete file)

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.

Blocks: 1512432
Attached 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.

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
Pushed by asurkov@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0c13462c5416
Remove toolkit handler binding, r=bgrins
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
QA Whiteboard: [qa-67b-p2]
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.