184899 2018-04-23 15:29:37 -0700 Investigate a better heuristic for when to prewarm a process 2018-05-09 10:51:59 -0700 1 1 1 Unclassified WebKit WebKit2 WebKit Nightly Build Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=184765 InRadar P2 Normal --- 1 saam saam achristensen beidson cdumez fpizlo ggaren msaboff rniwa webkit-bug-importer oldest_to_newest 1416994 0 saam 2018-04-23 15:29:37 -0700 Some ideas: - on the 'load' event - a timer X ms after the 'load' event. Or some combination of a timer an load event or first paint. Perhaps there are other heuristics as well 1417027 1 saam 2018-04-23 17:23:40 -0700 one way the current heuristic falls down is: - load "a.com" <-- does prewarm a process - press a link on "a.com" going to "b.com" <-- uses the prewarmed process - Go back (to "a.com" <--- does not prewarm a process - press a link on "a.com" for "c.com" <-- does not use a prewarmed process because one doesn't exist So we should take into account prewarming a process when using back/forward cache 1417028 2 saam 2018-04-23 17:26:17 -0700 (In reply to Saam Barati from comment #1) > one way the current heuristic falls down is: > - load "a.com" <-- does prewarm a process > - press a link on "a.com" going to "b.com" <-- uses the prewarmed process > - Go back (to "a.com" <--- does not prewarm a process > - press a link on "a.com" for "c.com" <-- does not use a prewarmed process > because one doesn't exist > > So we should take into account prewarming a process when using back/forward > cache actually, I'm off here. Maybe we don't need to do anything, since we do end up using a prewarmed process for "c.com", because "b.com" will prewarm a process 1422310 3 webkit-bug-importer 2018-05-09 09:52:08 -0700 <rdar://problem/40094910> 1422311 4 ggaren 2018-05-09 09:53:24 -0700 I think we also need to merge navigation process prewarming with the process prewarming we already do to speed up launch. 1422332 5 saam 2018-05-09 10:18:36 -0700 (In reply to Geoffrey Garen from comment #4) > I think we also need to merge navigation process prewarming with the process > prewarming we already do to speed up launch. Merge in what sense? They both call the same codepath if that’s what you mean 1422344 6 ggaren 2018-05-09 10:51:59 -0700 > Merge in what sense? They both call the same codepath if that’s what you mean Yeah, I guess that's what I mean. :P