"this should not be happening! arrgggggh!" repeatedly to console because gloda streaming a message that does not support the JS Mime Emitter
Categories
(MailNews Core :: Database, defect)
Tracking
(thunderbird115+ fixed)
People
(Reporter: ishikawa, Assigned: mkmelin)
References
()
Details
Attachments
(2 files)
(deleted),
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details |
(deleted),
text/x-phabricator-request
|
wsmwk
:
approval-comm-beta+
|
Details |
Comment 1•15 years ago
|
||
Comment 2•15 years ago
|
||
Reporter | ||
Comment 3•15 years ago
|
||
Reporter | ||
Comment 4•15 years ago
|
||
Comment 5•15 years ago
|
||
Comment 6•15 years ago
|
||
Comment 8•13 years ago
|
||
Comment 9•13 years ago
|
||
Comment 10•13 years ago
|
||
Comment 11•13 years ago
|
||
Comment 13•13 years ago
|
||
Updated•13 years ago
|
Comment 14•13 years ago
|
||
Updated•11 years ago
|
Updated•10 years ago
|
Comment 15•4 years ago
|
||
I do not know if this should or should not be happening, but it causes problems in web extensions during attempt to get e.g. more message headers for an nntp message, see Bug #1644027. The code even unable to properly reject a promise if some error happens. There are several places in the code where functions are not ready to actual argument with no message and exception is not caught, so the promise just is not resolved. The extension has no chance to be notified about the error.
Updated•2 years ago
|
Assignee | ||
Comment 16•1 years ago
|
||
Updated•1 years ago
|
Assignee | ||
Updated•1 years ago
|
Updated•1 years ago
|
Assignee | ||
Comment 17•1 years ago
|
||
Depends on D179933
Assignee | ||
Updated•1 years ago
|
Comment 18•1 years ago
|
||
Pushed by martin@humanoids.be:
https://hg.mozilla.org/comm-central/rev/d8c1f3ba2b40
Make NNTP streaming of messages work. r=leftmostcat
https://hg.mozilla.org/comm-central/rev/c357a7917dc6
Remove add-ons workaround for streaming nntp messages. r=john.bieling
Assignee | ||
Updated•1 years ago
|
Assignee | ||
Updated•1 years ago
|
Assignee | ||
Comment 19•1 years ago
|
||
Comment on attachment 9337656 [details]
Bug 545365 - Remove add-ons workaround for streaming nntp messages. r=john.bieling
[Approval Request Comment]
Regression caused by (bug #): never worked
User impact if declined: summary wouldn't work for news, so we'd have to add workarounds (like bug 1824422)
Testing completed (on c-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): fairly low risk
Assignee | ||
Comment 20•1 years ago
|
||
(Both patches)
Comment 21•1 years ago
|
||
Comment on attachment 9337656 [details]
Bug 545365 - Remove add-ons workaround for streaming nntp messages. r=john.bieling
[Triage Comment]
Approved for beta
Comment 22•1 years ago
|
||
Comment on attachment 9337435 [details]
Bug 545365 - Make NNTP streaming of messages work. r=leftmostcat
[Triage Comment]
Approved for beta
Comment 23•1 years ago
|
||
uplift |
Updated•1 year ago
|
Description
•