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

Issue 750854 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 322185
Owner: ----
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Report browser hangs to crash/

Project Member Reported by teravest@chromium.org, Jul 31 2017

Issue description

Chrome OS has a feature for detecting browser hangs.

session_manager periodically sends a D-Bus message to the browser process, and it must respond within a certain time, or the browser is killed (via SIGABRT).

We do get crash reports from chrome for the abort, but it's not obvious that they are triggered by a browser hang. Additionally, the instruction pointers could vary wildly for the same cause, so it's not possible to aggregate the crashes back for any hang statistics.

Kirtika suggested that we report browser hangs to crash/ much like kernel warnings, which seems like a good approach.

This has an additional benefit over reporting statistics in UMA in that we should be able to match up the Chrome-reported crash pretty easily by timestamp.

Any objections?
 

Comment 1 by derat@chromium.org, Aug 2 2017

Components: Internals>CrashReporting
Hmm, I want to say that (now-xoogler) cmasone@ was working on a mechanism at one point where session_manager could report a kill reason to Chrome's Breakpad instance via a named pipe, or something similar. I don't know if the work was ever completed, but it seems like it may be relevant here.

Is the proposal here to make session_manager report browser hangs so we can match those up with the Chrome SIGABRT reports? If so, I don't have any objections.
> Is the proposal here to make session_manager report browser hangs so we can match those up with the Chrome SIGABRT reports?

Yep, that's the idea.
so this is a dupe of issue 322185 ?
Mergedinto: 322185
Status: Duplicate (was: Untriaged)
Sounds good.

Sign in to add a comment