Closed
Bug 498363
Opened 15 years ago
Closed 15 years ago
closing stand-alone window causes (caught) js exception about shouldDeferMessageDisplayUntilAfterServerConnect
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 503624
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
(Blocks 1 open bug)
Details
(Keywords: regression, Whiteboard: [needs unit test])
Attachments
(1 obsolete file)
this isn't a function and js complains about this call:
http://mxr.mozilla.org/comm-central/source/mailnews/base/src/dbViewWrapper.js#1798
I suspect that means mark all read on folder exit isn't working...
Attachment #383280 -
Flags: superreview?(bugzilla)
Attachment #383280 -
Flags: review?(bugmail)
Assignee | ||
Updated•15 years ago
|
Blocks: gloda-ui-regressions
Comment 1•15 years ago
|
||
nit: You added an extra space.
This needs a unit test (or the feature should be removed). Since the error is safely contained in a catch block, I think the fix can wait for its unit test.
The test could either go in the DBViewWrapper xpcshell tests or as a mozmill test.
Flags: in-testsuite?
Whiteboard: [needs unit test]
Updated•15 years ago
|
Comment 2•15 years ago
|
||
Comment on attachment 383280 [details] [diff] [review]
proposed fix
r-'ing to get this off my request queue until there's a unit test or the feature is removed.
Attachment #383280 -
Flags: review?(bugmail) → review-
Comment 3•15 years ago
|
||
Comment on attachment 383280 [details] [diff] [review]
proposed fix
cancelling per asuth's comments.
Attachment #383280 -
Flags: superreview?(bugzilla)
Assignee | ||
Comment 4•15 years ago
|
||
Comment on attachment 383280 [details] [diff] [review]
proposed fix
this wouldn't have quite fixed it - the fix is in bug 503624
Attachment #383280 -
Attachment is obsolete: true
Assignee | ||
Comment 5•15 years ago
|
||
resolving as dup of user-facing problem.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
Updated•11 years ago
|
Flags: in-testsuite?
You need to log in
before you can comment on or make changes to this bug.
Description
•