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)

defect
Not set
normal

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)

Attached patch proposed fix (obsolete) (deleted) — Splinter Review
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)
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]
Keywords: regression
OS: Windows Vista → All
Hardware: x86 → All
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 on attachment 383280 [details] [diff] [review] proposed fix cancelling per asuth's comments.
Attachment #383280 - Flags: superreview?(bugzilla)
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
resolving as dup of user-facing problem.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → DUPLICATE
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: