Open
Bug 1250472
Opened 9 years ago
Updated 2 years ago
[e10s] browser.xml and remote-browser.xml shouldn't individually add observers for browser:purge-session-history
Categories
(Firefox :: Tabbed Browser, defect, P3)
Tracking
()
NEW
Tracking | Status | |
---|---|---|
e10s | + | --- |
People
(Reporter: Gijs, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: perf)
Because all they're doing is then converting it into a message that triggers purging the session history in browser-content.js Instead, there should just be one observer (nsBrowserGlue.js seems to already do this and would probably do nicely) that broadcasts the async message once to every process, so it gets picked up in every browser-content.js. Then we also don't need to worry about removing the observer in the destroy() method, which doesn't always get called (see bug 1249439). It also looks like the starting time gets lost and we just purge all the history out of the browsers. Not sure if that can easily be fixed when fixing this, but it would be a nice bonus.
Updated•9 years ago
|
tracking-e10s:
--- → +
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•