Closed
Bug 535519
Opened 15 years ago
Closed 10 years ago
session restore is overly event hungry
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 28
People
(Reporter: dietrich, Assigned: ttaubert)
References
Details
(Keywords: perf, Whiteboard: [Snappy:P3])
we register for a bunch of events, which hurts page load time, especially on mobile. we need to investigate ways to reduce the number of events we listen to, as well as increasing the interval between actually doing anything about oft-recurring events.
Reporter | ||
Updated•15 years ago
|
Whiteboard: [tsnap]
Reporter | ||
Updated•13 years ago
|
Whiteboard: [tsnap] → [snappy]
Updated•13 years ago
|
Whiteboard: [snappy] → [snappy:p4]
Reporter | ||
Comment 1•13 years ago
|
||
Global notifications: "domwindowopened", "domwindowclosed", "quit-application-requested", "quit-application-granted", "browser-lastwindow-close-granted", "quit-application", "browser:purge-session-history", "private-browsing", "browser:purge-domain-data", "private-browsing-change-granted" Events: window: load tabbrowser: TabOpen, TabClose, TabSelect, TabShow, TabHide, TabPinned, TabUnpinned tab: load, pageshow, change, input, DOMAutoComplete textarea in design mode: keypress
Updated•13 years ago
|
Whiteboard: [snappy:p4] → [Snappy:P3]
Reporter | ||
Updated•12 years ago
|
Blocks: sessionRestoreJank
Comment 2•11 years ago
|
||
Can we use DeferredTask.jsm for some of the oft-recurring events such as keypress?
Comment 3•10 years ago
|
||
We should be able to call this fixed by bug 930967 and friends, I think?
Assignee | ||
Comment 4•10 years ago
|
||
Yeah, I don't think we listen for too many events. It's just the right number to invalidate data. However, this bug also mentions mobile that like every product has their own sessionstore implementation. WRT to desktop Firefox I don't see how listening to events hurts perf as long as we make sure to not collect data immediately and have a small damper. All this is done now that we broadcast data from frame scripts. Let's call this fixed and file a new, more specific bug if we think there is room for improvement in sessionstore's event handling.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Assignee: nobody → ttaubert
Target Milestone: --- → Firefox 28
You need to log in
before you can comment on or make changes to this bug.
Description
•