WebView crashes don't indicate if multiprocess is in use |
|||
Issue descriptionChrome crashes normally indicate the process type which crashed - WebView microdumps don't include this information, and so there's no way to easily tell (or filter on) whether a crash was in single process or multiprocess mode, and if multiprocess, whether it was a browser or renderer crash. We should probably include this information in the microdump so that the crash frontend can expose it and allow easy filtering (so that we can spot multiprocess-specific issues, or ignore multiprocess-mode crashes when looking at general release stability). We may want to just try to expose the same process type information that Chrome uses, so that it can be handled the same way? It's possible to tell this *indirectly* in some cases: e.g. the process name is a big hint to see if it's a renderer crash, but it's not easy to filter by that, and there's not really any way to tell the difference between a browser crash in single vs multiprocess mode.
,
Jun 3 2016
Looked a bit: I guess we need an extra field "is_webview_multiprocess" in MicrodumpExtraInfo?
,
Jun 3 2016
And it is better that we somehow make that extra field not appear in microdump for clank, I guess
,
Jun 3 2016
It seems like it'd be more useful to have the process type, and not just a boolean - renderer vs browser vs singleprocess?
,
Jul 26 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e1a6b0a706a4badbe6131215da9e335d62ac5c93 commit e1a6b0a706a4badbe6131215da9e335d62ac5c93 Author: hush <hush@chromium.org> Date: Tue Jul 26 19:00:23 2016 Dump process type into breakpad. The process type can be "webview", "browser", or "renderer". BUG= 616774 Review-Url: https://codereview.chromium.org/2086483006 Cr-Commit-Position: refs/heads/master@{#407869} [modify] https://crrev.com/e1a6b0a706a4badbe6131215da9e335d62ac5c93/android_webview/lib/main/aw_main_delegate.cc [modify] https://crrev.com/e1a6b0a706a4badbe6131215da9e335d62ac5c93/components/crash/content/app/breakpad_linux.cc [modify] https://crrev.com/e1a6b0a706a4badbe6131215da9e335d62ac5c93/components/crash/content/app/breakpad_linux.h
,
Jul 26 2016
,
Aug 24 2016
,
Aug 24 2016
see internal bug b/29610934 |
|||
►
Sign in to add a comment |
|||
Comment 1 by hush@chromium.org
, Jun 3 2016Status: Assigned (was: Available)