Remove toolkit handler binding

RESOLVED FIXED in Firefox 67

Status

()

task
P3
normal
RESOLVED FIXED
5 months ago
10 days ago

People

(Reporter: ntim, Assigned: surkov)

Tracking

(Blocks 1 bug)

unspecified
mozilla67
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox67 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Reporter

Description

5 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

5 months ago
Blocks: 1512432
Reporter

Comment 1

5 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

4 months 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

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

Comment 9

4 months ago
bugherder
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
QA Whiteboard: [qa-67b-p2]
Reporter

Updated

10 days ago
Type: enhancement → task
You need to log in before you can comment on or make changes to this bug.