Init safe browsing later during startup
Categories
(Toolkit :: Safe Browsing, enhancement, P3)
Tracking
()
People
(Reporter: alexical, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Keywords: perf:startup)
Attachments
(1 file)
We read around 7MB from the goog-phish-proto.vlpset
file during startup, before about:home has finished loading. It seems like this could be pushed back.
Comment 1•5 years ago
|
||
Marking fxperf:p3 for now because I'm not sure we could push this back much (I would guess we need it as close as possible to the first network request), so I'm not sure we could gain much.
I think it's initialized at https://searchfox.org/mozilla-central/rev/4d2a9d5dc8f0e65807ee66e2b04c64596c643b7a/browser/components/BrowserGlue.jsm#2073-2080 and there's a comment there saying it needs to be initialized early-ish.
I could be convinced it's higher priority though, especially if you have an idea of where/when we could initialize this instead.
Reporter | ||
Comment 2•5 years ago
|
||
It would need to happen at the same time as it currently does if we're not just loading about:home. But if we are, it would ideally happen once we know about:home has settled. I believe we know about:home has settled around the time we record this timestamp. It would be a bit awkward including it there, but that's about where in the timeline I think it should go. We should also race against that with a callback in _scheduleArbitrarilyLateIdleTasks
in case about:home never settles for some reason.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 3•5 years ago
|
||
Updated•3 years ago
|
Updated•2 years ago
|
Updated•7 months ago
|
Description
•