Disable registerContentHandler() in stable

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P3
normal
RESOLVED FIXED
Last year
4 months ago

People

(Reporter: jkt, Assigned: jkt)

Tracking

({dev-doc-complete, site-compat})

60 Branch
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

Last year
As outlined in the intent to unship (https://groups.google.com/forum/#!topic/mozilla.dev.platform/jeTDLz38_RE/discussion) we should remove the code from stable. This bug is to disable the code in stable for now.

Comment 2

Last year
Should cleaning up after bug 1429732 be included in this bug, or in that bug, or a new bug?
Assignee

Comment 3

Last year
I'll make a new bug for that, in my super tired state I made this patch message have the wrong function name. We should remove the actual code after 62 is stable imo just to ensure we don't break anything.
Comment hidden (mozreview-request)
Assignee

Updated

Last year
Summary: Remove registerContentHandler() code in stable → Disable registerContentHandler() in stable
Priority: -- → P3

Comment 6

Last year
mozreview-review
Comment on attachment 8974603 [details]
Bug 1460481 - Disable registerContentHandler from stable releases.

https://reviewboard.mozilla.org/r/242966/#review249296
Attachment #8974603 - Flags: review?(dao+bmo) → review+

Comment 7

Last year
Pushed by jkingston@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bb0bb62a43e9
Disable registerContentHandler from stable releases. r=dao

Comment 9

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/bb0bb62a43e9
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
The article has been updated slightly, and I’ve submitted PR #2440 to update the BCD database to indicate this change.

This is also listed on Firefox 62 for developers.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.