Closed
Bug 1641573
Opened 5 years ago
Closed 5 years ago
Compose scripts: messenger.* namespace is undefined
Categories
(Thunderbird :: Add-Ons: Extensions API, defect)
Tracking
(thunderbird_esr78 fixed, thunderbird79 wontfix)
RESOLVED
FIXED
Thunderbird 80.0
People
(Reporter: tdulcet, Assigned: darktrojan)
Details
Attachments
(1 file)
|
4.59 KB,
patch
|
mkmelin
:
review+
wsmwk
:
approval-comm-esr78+
|
Details | Diff | Splinter Review |
In Compose scripts, the messenger.* namespace/variable is undefined. If you try to use it, you get ReferenceError: messenger is not defined in the console. The browser namespace works fine.
|
Assignee | |
Updated•5 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
|
Assignee | |
Comment 1•5 years ago
|
||
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9160242 -
Flags: review?(mkmelin+mozilla)
|
||
Comment 2•5 years ago
|
||
Comment on attachment 9160242 [details] [diff] [review]
1641573-compose-scripts-messenger-1.diff
Review of attachment 9160242 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM, r=mkmelin
Attachment #9160242 -
Flags: review?(mkmelin+mozilla) → review+
|
|
Assignee | |
Updated•5 years ago
|
Keywords: checkin-needed-tb
Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/9dc474fda8ab
Add messenger alias to compose scripts. r=mkmelin DONTBUILD
|
|
||
Updated•5 years ago
|
status-thunderbird79:
--- → affected
status-thunderbird_esr78:
--- → affected
Target Milestone: --- → Thunderbird 80.0
|
Reporter | |
Comment 4•5 years ago
|
||
Can this please be uplifted to 78 ESR?
|
|
Assignee | |
Comment 5•5 years ago
|
||
Comment on attachment 9160242 [details] [diff] [review]
1641573-compose-scripts-messenger-1.diff
For extensions only, has a test.
Attachment #9160242 -
Flags: approval-comm-esr78?
|
|
Assignee | |
Updated•5 years ago
|
|
||
Comment 6•5 years ago
|
||
Comment on attachment 9160242 [details] [diff] [review]
1641573-compose-scripts-messenger-1.diff
Approved for esr78
Attachment #9160242 -
Flags: approval-comm-esr78? → approval-comm-esr78+
|
|
Assignee | |
Comment 7•5 years ago
|
||
| bugherder uplift | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•