Closed Bug 535408 Opened 10 years ago Closed 10 years ago

Increase Sessionstore interval to 15s

Categories

(Firefox :: Session Restore, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 3.7a1

People

(Reporter: zpao, Assigned: zpao)

References

Details

(Whiteboard: [tsnap])

Attachments

(2 files)

Per discussion with dietrich & mconnor, changing the default interval to 15s and not saving for readonly events will significantly reduce the number of writes to disk we do (33+% fewer) without any real loss of functionality.
Whiteboard: [tsnap]
Attachment #418232 - Flags: review?(dietrich) → review+
Pushed http://hg.mozilla.org/mozilla-central/rev/314597526cbf
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a1
Hmm, I forgot to change the default value in nsSessionStore.js. It doesn't really matter though since the pref is read at initialization, but we should probably keep those in sync.
Change the default interval in nsSessionStore.js too (in case the pref is deleted from firefox.js)
Attachment #421492 - Flags: review?(dietrich)
Attachment #421492 - Flags: review?(dietrich) → review+
Verified by checking nsSessionStore.js, about:config, and firefox.js.
Status: RESOLVED → VERIFIED
Blocks: 558649
You need to log in before you can comment on or make changes to this bug.