Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linux Crash doesn't list in chrome://crashes. Navigating to chrome://crash shows error message #1251

Closed
srirambv opened this issue Sep 24, 2018 · 8 comments · Fixed by brave/brave-core#1480
Assignees
Labels

Comments

@srirambv
Copy link
Contributor

Description

Linux Crash doesn't list in chrome://crashes. Navigating to chrome://crash shows error message

Steps to Reproduce

  1. Open a new tab and visit chrome://crash
  2. Open a new tab and visit chromes://crashes/ nothings listed
  3. After a min the GTK crash message pops up

Actual result:

image

Expected result:

Should list the crash on browser and send the report if its enabled in settings

Reproduces how often:

100%

Brave version (chrome://version info)

Brave 0.55.6 Chromium: 70.0.3538.16 (Official Build) dev (64-bit)
Revision 16ed95b41bb05e565b11fb66ac33c660b721f778-refs/branch-heads/3538@{#306}
OS Linux

Reproducible on current release:

Yes

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Chrome?

Additional Information

Follow up to #285 for Linux

@srirambv srirambv added this to the 1.x Backlog milestone Sep 24, 2018
@rebron rebron added the priority/P2 A bad problem. We might uplift this to the next planned release. label Sep 28, 2018
@rebron rebron modified the milestones: 1.x Backlog, 0.56.x - Beta Oct 16, 2018
@pilgrim-brave pilgrim-brave removed their assignment Oct 19, 2018
@rebron rebron modified the milestones: 0.56.x - Beta, 0.57.x - Dev Oct 23, 2018
@rebron rebron removed this from the 0.57.x - Dev milestone Oct 30, 2018
@bbondy bbondy added this to the 1.x Backlog milestone Oct 30, 2018
@kjozwiak
Copy link
Member

@bbondy @rebron we fix this in 0.60.x and finally get crash reporting working under Linux. Win 7 reporting will be fixed under 0.60.x via #1188 so we might as well try getting this into the same build 👍

@mkarolin mkarolin self-assigned this Jan 28, 2019
mkarolin added a commit to brave/brave-core that referenced this issue Jan 28, 2019
- Patches ChromeCrashReporterClient::GetCollectStatsConsent to check for
  BRAVE_CHROMIUM_BUILD in addition to GOOGLE_CHROME_BUILD.
- Patches breakpad Linux implementation with Brave's crash reporting upload
  endpoint.

Fixes brave/brave-browser#1251
mkarolin added a commit to brave/brave-core that referenced this issue Jan 30, 2019
- Patches ChromeCrashReporterClient::GetCollectStatsConsent to check for
  BRAVE_CHROMIUM_BUILD in addition to GOOGLE_CHROME_BUILD.
- Patches breakpad Linux implementation with Brave's crash reporting upload
  endpoint.

Fixes brave/brave-browser#1251
mkarolin added a commit to brave/brave-core that referenced this issue Jan 30, 2019
- Patches ChromeCrashReporterClient::GetCollectStatsConsent to check for
  BRAVE_CHROMIUM_BUILD in addition to GOOGLE_CHROME_BUILD.
- Patches breakpad Linux implementation with Brave's crash reporting upload
  endpoint.

Fixes brave/brave-browser#1251
mkarolin added a commit to brave/brave-core that referenced this issue Jan 30, 2019
- Patches ChromeCrashReporterClient::GetCollectStatsConsent to check for
  BRAVE_CHROMIUM_BUILD in addition to GOOGLE_CHROME_BUILD.
- Patches breakpad Linux implementation with Brave's crash reporting upload
  endpoint.

Fixes brave/brave-browser#1251
@kjozwiak
Copy link
Member

Went through the test case outlined in the PR using Mint 19.1 x64 VM and everything seems to be working. Once I enabled crash reporting via the brave://settings, I made sure that crashes where appearing under brave://crashes when crashing via:

  • brave://crash and ensured that the crash was recorded under brave://crashes and was visible in stats.brave
  • chrome://gpucrash and ensured that the crash was recorded under brave://crashes and was visible in stats.brave

However, there's one difference that I noticed. Under Windows/macOS, even if you have crash reporting disabled, the crashes are saved in chrome://crashes but are not sent. The user has the ability to send each one manually without needing to enable reporting. However, under Linux, if you have crash reporting disabled, crash reports will not be recorded in brave://crashes.

The other ting that's a bit confusing is that on Linux, the setting appears as:

Automatically send usage statistics and crash reports to Brave Software

However, on Windows and macOS, the setting appears as:

Automatically send crash reports to Brave

I'll create an issue re: the wording discrepancy. @rebron @bbondy thoughts on changing the crash behaviour so it matches macOS/Win? Or leaving it like this is fine?

@srirambv
Copy link
Contributor Author

Verified the steps form test plan on Ubuntu (physical machine) and works great. Was also able to verify the crash report on stats.brave
image

Under Windows/macOS, even if you have crash reporting disabled, the crashes are saved in chrome://crashes but are not sent.

Agree with @kjozwiak behaviour is different compared to mac/Windows when crash reporting is disabled. Should be similar.

@kjozwiak
Copy link
Member

@mkarolin it doesn't seem like the reports are being uploaded into stats.brave.com. They're appearing under brave://crashes, however it doesn't find anything under stats.brave when searching using the generated crash id. Example:

  • launch 0.60.44 Chromium: 72.0.3626.109
  • enable crash reporting via brave://settings and restart the browser
  • once restarted, crash using brave://crash or chrome://gpucrash

You'll see the following:

--2019-02-19 16:08:18--  https://laptop-updates.brave.com/1/bc-crashes
Resolving laptop-updates.brave.com (laptop-updates.brave.com)... 151.101.125.7
Connecting to laptop-updates.brave.com (laptop-updates.brave.com)|151.101.125.7|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 16 [text/html]
Saving to: ‘/dev/fd/4’

     0K                  
 Crash dump id: 49e4d4f7cab8d55a

However, if you search stats.brave for 49e4d4f7cab8d55a, you won't find the crash. Tried it a few times on two different linux machines with the same results. Platforms used:

  • Mint 19.1 x64 (psychical machine) - didn't upload to stats.brave
  • Ubuntu 18.04 x64 (VM) - didn't upload to stats.brave

Here are some of the crash reports that I was expecting to find under stats.brave:

  • 1cfc6d359217fb1d
  • 11b7fde0ca05667a
  • 413719a15012e33b

@mkarolin
Copy link
Contributor

@kjozwiak, @srirambv: regarding the difference in behavior between Win/Mac and Linux, that is likely because Win/Mac use Crashpad for crash handling and reporting, whereas Linux uses Breakpad. There's some code that indicates that there's an effort to transition Linux to Crashpad as well, but it hasn't happened yet (https://chromium.googlesource.com/crashpad/crashpad/+/HEAD/doc/status.md).

@kjozwiak
Copy link
Member

@mkarolin awesome, sounds good 👍

Regarding the above issue with crashes not being uploaded. @btlechowski confirmed that he's not seeing any crashes uploaded on Win 7. I tried Windows 10 x64 and getting the same results, not seeing any of the crashes appear in stats.brave. @LaurenWags mentioned that it's working for macOS. So we either regressed reporting in the past ~week or so or there's an issue with the server. I pinged @aekeus and he's going to take a look when he gets some time 👍 If the server side looks fine, I'll try getting a regression range.

@srirambv
Copy link
Contributor Author

@kjozwiak the crash report id that i mentioned in #1251 (comment) is still visible on stats.brave. However on the latest rc build when i force crash and generate the id(6fea8799f09804d3) that is not being uploaded to stats. So its either going into backlog and still not updated on stats or its broken after channel migration happened because #1251 (comment) was done when it was still on beta channel

@kjozwiak
Copy link
Member

@srirambv yup, just waiting for a confirmation from @aekeus. Once we know that the servers are not having issues nor are backed up, we'll need go through a regression range and find out where reporting broke.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants