Localstore not being saved in mail

VERIFIED WORKSFORME

Status

()

Core
RDF
P3
normal
VERIFIED WORKSFORME
19 years ago
18 years ago

People

(Reporter: scottputterman, Assigned: Chris Waterson)

Tracking

Trunk
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
In today's build, when I change the expand/collapse state of a folder or change 
the threaded state of a folder, it is not being saved when I quit.  I can see 
that localstore.rdf is not being modified.  This is only happening when I run 
-mailnews.  If I startup the browser then it is being saved.

I think this might be happening due to a leak in ~nsXULDocument because I can 
see that LocalStoreImpl::Flush() is never being called.  Assuming this is the 
right code, I wonder if we need some way to save the local store without relying 
on a destructor?  Possibly doing it onshutdown.

Comment 1

19 years ago
I agree that doing this in an OnShutdown type method would be good. But when I 
run, I am hitting the destructor for the nsXULDocument - I'm just running mail, 
and not doing anything fancy (like maybe even reading a message - I should try 
that).
(Assignee)

Comment 2

19 years ago
Better yet, you could do it in messenger.xul's "onunload" handler. I think that 
the browser does it this way.

Should we try to find the leak? (Or at least file a different bug for that?)
(Assignee)

Comment 3

19 years ago
wait, you said you *are* hitting nsXULDocument's dtor? mysterious.
Status: NEW → ASSIGNED
Target Milestone: M15

Comment 4

19 years ago
I'm not seeing Putterman's problem at all - but that's only fair, he's not seeing all my problems either.
(Assignee)

Comment 5

18 years ago
putterman: is this still broken?
(Reporter)

Comment 6

18 years ago
I haven't seen it recently.  Marking WORKSFORME.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → WORKSFORME

Comment 7

18 years ago
Verified
Platform: PC
OS: Windows 98
Mozilla Build: 2000101020 M18 Trunk Build
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.