Closed Bug 1614237 Opened 4 years ago Closed 4 years ago

Remove legacy API

Categories

(Thunderbird :: Add-Ons: General, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 74.0

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file, 1 obsolete file)

We don't intend to still have the legacy API at the next ESR, and the time has come to remove it.

Attached patch 1614237-remove-legacy-1.diff (obsolete) — Splinter Review

This removes the API but I think it would be helpful to have all extensions using the API disabled automatically somehow. The best way would be for authors to mark their extensions as incompatible after 73, but that is never going to happen. We could mark them all as incompatible on ATN but I'm not sure if that would actually result in Thunderbird disabling them (I think we should do it anyway). I'm going to look at some ways we could do it in Thunderbird itself.

Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9125376 - Flags: review?(mkmelin+mozilla)

Found some more things to remove.

Attachment #9125376 - Attachment is obsolete: true
Attachment #9125376 - Flags: review?(mkmelin+mozilla)
Attachment #9125377 - Flags: review?(mkmelin+mozilla)
Comment on attachment 9125377 [details] [diff] [review]
1614237-remove-legacy-2.diff

Review of attachment 9125377 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM, I'm sure we'll find even more eventually. r=mkmelin
Attachment #9125377 - Flags: review?(mkmelin+mozilla) → review+

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/3f3fc2c0d804
Remove legacy extensions API. r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

I decided to push this without any extension disabling mechanism for now, so that there's at least one Daily build for users before the code moves to comm-beta.

Target Milestone: --- → Thunderbird 74.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: