New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 893821 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Dec 11
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-12-13
OS: Mac
Pri: 1
Type: Bug

Blocking:
issue 859152



Sign in to add a comment

Mac PWAs: Report crashes of the AppShim process

Project Member Reported by ccameron@chromium.org, Oct 9

Issue description

We get no crash reports for the AppShim process in chromecrash/ (as far as I can tell). Since we are now doing a fair amount of work in this process, we should make sure that we are gathering crash reports from this process.

For background on the AppShim process, see this document
https://docs.google.com/document/d/1ps3-_5RFQu37x1fuYY79Z5_WnpPPFqjGXksxZDahgiI/edit?usp=sharing
Or ChromeAppModeStart_v4:
https://cs.chromium.org/chromium/src/chrome/app_shim/chrome_main_app_mode_mac.mm?rcl=8a6cc166e9b9856c151493d807c080774200c126&l=588

What is necessary to start generating reports from this process?
 
ChromeAppModeStart_v4 needs to funnel itself through crash reporter initialization.

ChromeMainDelegate::InitMacCrashReporter() (chrome/app/chrome_main_delegate.cc) is where we normally do this initialization on macOS. That’s called by ChromeMainDelegate::PreSandboxStartup(). I guess ChromeAppModeStart_v4() isn’t coming through ChromeMain() at all?
True, we don't come through ChromeMain.

We are also not a child process -- we are a peer -- we dlopen() chrome, and we establish a mojo connection.

Simply adding the lines
  crash_reporter::SetCrashReporterClient(new ChromeCrashReporterClient);
  crash_reporter::InitializeCrashpad(true /* initial_client */, "" /* process_type */);
seems to not crash at least. Not sure it actually sets things up appropriately. If I set process_type to something like "appshim" then it's not happy.

Is there any sort of testing I can do to see if that works? IIUC, this would report all appshim crashes as browser crashes (which we would want to differentiate somehow).
Labels: proj-MacPwa
Components: UI>Browser>WebAppInstalls
Labels: OS-Mac
Labels: -proj-MacPwa
Owner: sdy@chromium.org
Status: Assigned (was: Available)
Labels: -Pri-3 M-73 Pri-1
Project Member

Comment 8 by bugdroid1@chromium.org, Dec 11

NextAction: 2018-12-13
Status: Fixed (was: Assigned)
Should be all set, setting a reminder to verify.
The NextAction date has arrived: 2018-12-13
Status: Verified (was: Fixed)

Sign in to add a comment