Closed
Bug 347554
Opened 18 years ago
Closed 18 years ago
compose window recycling broken, exception terminates gComposeRecyclingListener.onClose
Categories
(Thunderbird :: Message Compose Window, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 282669
Thunderbird2.0
People
(Reporter: KaiE, Assigned: mscott)
Details
Attachments
(1 file)
833 bytes,
patch
|
Details | Diff | Splinter Review |
Linux. Mozilla 1.8 branch. Thunderbird. Mail Message Compose Window Recycling is broken. An exception happens during execution of gComposeRecyclingListener.onClose This causes function to terminate without completing. This causes at least bug 345784, where the encryption/signing flags can no longer be controlled in follow up messages. I expect this causes other failures in recycled compose windows as well. The exact message I get is: Error: top.awInputElement has no properties Source File: chrome://messenger/content/messengercompose/MsgComposeCommands.js Line: 196 The Error message and source file is correct. I was able to "hide" this bug by wrapping access to this code if (top.awInputElement.getAttribute('focused') != '') top.awInputElement.removeAttribute('focused'); in a try/catch block (see also attached patch). It was confusing that the error message lists lines number 196, although this code is really located at lines 232/233. (This incorrect line number might be another bug in JS error reporting???)
Reporter | ||
Comment 1•18 years ago
|
||
Not sure if this is the right solution to the problem, but at least this patch hides the bug.
Attachment #232349 -
Flags: review?(mscott)
Reporter | ||
Comment 2•18 years ago
|
||
Thanks to Neil who told me, this is a duplicate of bug 282669. The patch in that bug will be checked in now. *** This bug has been marked as a duplicate of 282669 ***
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 3•18 years ago
|
||
Comment on attachment 232349 [details] [diff] [review] Patch v1 - hides problem canceling a review request, I don't think we need this anymore.
Attachment #232349 -
Flags: review?(mscott)
You need to log in
before you can comment on or make changes to this bug.
Description
•