about:newaddon is broken

RESOLVED FIXED in Thunderbird 63.0

Status

defect
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: darktrojan, Assigned: darktrojan)

Tracking

Trunk
Thunderbird 63.0

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

When a new addon is detected, Thunderbird should open a tab at about:newaddon?id=xyz@abc, but instead the "The address isn't valid" page is shown.
This was removed in bug 1331521 three month ago in Add-ons Manager. Do we have code that points to this page?
Try DXR? https://dxr.mozilla.org/comm-central/search?q=about%3Anewaddon&redirect=false

Yields:
mail/components/mailGlue.js
191 { contentPage: "about:newaddon?id=" + aAddon.id,

Fix welcome ;-) - Fortunately not affecting TB 60.
I was about to post a patch removing the code which loads this non-existent page, but perhaps it would be better to restore the non-existent page instead, given that:

* Most of our extensions are going to continue to be legacy extensions for the time being.
* Creating a UI for webextension permissions is going to be difficult. Perhaps about:newaddon would even be a good starting point.
Reinstates about:newaddon, as it was before removal in https://hg.mozilla.org/mozilla-central/rev/c85fefa9248c . I've made some changes to handle our legacy extensions.
Assignee: nobody → geoff
Status: NEW → ASSIGNED
These are the differences between the code that was removed from m-c, and the code I'm adding to c-c.
See Also: → 1383560
Comment on attachment 8995024 [details] [diff] [review]
1473178-about-newaddon-2.diff

I don't know who should review this. Feel free to pass it on.
Attachment #8995024 - Flags: review?(jorgk)
Comment on attachment 8995025 [details] [diff] [review]
1473178-about-newaddon-diff.diff

This patch looks somewhat less frightening, however, for a qualified review you'd need to understand all the bits, like what's the story with the listener, etc. Philipp would be the only reviewer who knows this stuff. So would you like to pile it into his queue?
Comment on attachment 8995024 [details] [diff] [review]
1473178-about-newaddon-2.diff

I couldn't win Aceman, so it will need to be Philipp.
Attachment #8995024 - Flags: review?(jorgk) → review?(philipp)
Comment on attachment 8995024 [details] [diff] [review]
1473178-about-newaddon-2.diff

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

Code looks fine. I'm not really happy about resurrecting removed code, but it seems this is reasonable.
Attachment #8995024 - Flags: review?(philipp) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/7a59c0ecec8e
Reinstate about:newaddon. r=philipp DONTBUILD
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 63.0
Component: General → Add-Ons: General
You need to log in before you can comment on or make changes to this bug.