Intermittent comm/mail/components/extensions/test/browser/browser_ext_tabs_events.js | undefined - Expected: onCreated, Actual: onUpdated -
Categories
(Thunderbird :: Add-Ons: Extensions API, defect, P5)
Tracking
(thunderbird_esr78 wontfix)
Tracking | Status | |
---|---|---|
thunderbird_esr78 | --- | wontfix |
People
(Reporter: intermittent-bug-filer, Assigned: darktrojan)
References
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
Filed by: geoff [at] darktrojan.net
Parsed log: https://treeherder.mozilla.org/logviewer.html#?job_id=314542480&repo=comm-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/CY4y3UmXSDqjL7a2bL8hbw/runs/0/artifacts/public/logs/live_backing.log
This is not bug 1657061.
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
TabProgressListener QIs to nsISupportsWeakReference, which means the instances we add only survive
until the next GC. Also if we change the layout of the 3-pane window, the progress listener on
#messagepane disappears.
Once all that is fixed we can more reliably test events on the tabs API.
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/0af9452aeb51
Keep a reference to TabProgressListeners and improve testing of tabs.onUpdated. r=mkmelin
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 4•4 years ago
|
||
Comment on attachment 9173538 [details]
Bug 1662395 - Keep a reference to TabProgressListeners and improve testing of tabs.onUpdated. r?mkmelin
[Approval Request Comment]
User impact if declined: some WebExt events don't work correctly
Testing completed (on c-c, etc.): on beta since 82
Risk to taking this patch (and alternatives if risky): there's some risk but I haven't seen any problems other than bug 1662912 which also should be uplifted
Assignee | ||
Comment 5•4 years ago
|
||
Comment on attachment 9173538 [details]
Bug 1662395 - Keep a reference to TabProgressListeners and improve testing of tabs.onUpdated. r?mkmelin
Scratch that. The platform changes that happened in 79 to make it not worthwhile backporting.
Assignee | ||
Updated•4 years ago
|
Description
•