search

brave / brave-browser

Brave browser for Android, iOS, Linux, macOS, Windows.
https://brave.com
Mozilla Public License 2.0
17.85k stars 2.33k forks source link

[Perf] investigate changing in ChildProcess.Launched.UtilityProcessHash#count #39864

Open atuchin-m opened 3 months ago

atuchin-m commented 3 months ago

https://brave-perf-dashboard.appspot.com/report?sid=413692be7410b8b7c7d6b0ac7d3101e7377fc1071fbad83d7f8c569634c3a29e&rev=20243

Alert information:
Median of segment before: 6
Median of segment after: 8
Relative change: 33.3%
Test: BravePerf/win11-hp-laptop-typical/loading.desktop.brave/ChildProcess.Launched.UtilityProcessHash#count/warm/IGN_warm
Value: 8.0
image
atuchin-m commented 3 months ago

The process number of changed 8 => 6 => 8. So now we got the same value we had before. The number of long-living processes are unchanged.

The commit ranges match to https://github.com/brave/brave-core/commit/c24b0ac23834475dec80b10b291f5bde7b524f23.

It's likely that MeticsService is in charge of +2 extra short-living utility processes. We need to find why and whether it makes sense for Brave.