diff options
| author | 2022-10-15 01:42:59 +0000 | |
|---|---|---|
| committer | 2022-10-17 19:04:35 +0000 | |
| commit | 5e5de12eee429e5fbf21369b27de1c41221eda92 (patch) | |
| tree | 98f5871f02e619019a9fb788b5538ec08b3df5c5 | |
| parent | cb23441ce5323bc30c7729d2cc478a99ecfe274c (diff) | |
Update IsZygote to return true for webview_zygote
This change adds "webview_zygote" to the list of processes that the
IsZygote() function will return true for. This is intended to reduce the
time spent during the "DUMP TRACES" section of dumpstate, which has been
seen to always time out on the WebViewZygoteInit process.
Over two trials of health/bugreport/capturebugreport on bluejay, this
was seen to have reduced the duration of "DUMP TRACES" by 7.474s
(-35.95%) and the duration of the entire bug report by 6.939s (-8.32%).
Test: Build and flash; verify that health/bugreport/capturebugreport
demonstrates that
1) the WebViewZygoteInit process is no longer captured by
libdebuggerd_client
2) the duration of the dump traces section is shorter
Bug: 253670403
Change-Id: Ib1ea2f16c7df97181e98d0723d5ed25415f2d48f
| -rw-r--r-- | libs/dumputils/dump_utils.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/dumputils/dump_utils.cpp b/libs/dumputils/dump_utils.cpp index a6585c5ad2..067ce17c33 100644 --- a/libs/dumputils/dump_utils.cpp +++ b/libs/dumputils/dump_utils.cpp @@ -208,5 +208,5 @@ bool IsZygote(int pid) { cmdline = std::string(cmdline.c_str()); return cmdline == "zygote" || cmdline == "zygote64" || cmdline == "usap32" || - cmdline == "usap64"; + cmdline == "usap64" || cmdline == "webview_zygote"; } |