Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(31)

Issue 5303002: Add more exception information to breakpad. (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
CC:
chromium-reviews, M-A Ruel
Visibility:
Public.

Description

Add more exception information to breakpad. When an HTTP exception information is logged, no information is sent by default. Add an helper function to manually extract interesting information to append it to the exception type. Also disable another pylint warning. TEST=manual test BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=67306

Patch Set 1 #

Patch Set 2 : Add url too #

Patch Set 3 : Add info() #

Patch Set 4 : Add back support for numeric IP host #

Patch Set 5 : remove upload.py added by error, limit stack length to 4096 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -11 lines) Patch
M breakpad.py View 1 2 3 4 2 chunks +30 lines, -8 lines 0 comments Download
M gclient_utils.py View 1 chunk +0 lines, -2 lines 0 comments Download
M pylintrc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
10 years, 1 month ago (2010-11-23 01:54:42 UTC) #1
M-A Ruel
(John is on vacation)
10 years, 1 month ago (2010-11-24 20:04:56 UTC) #2
nsylvain
10 years, 1 month ago (2010-11-24 20:38:44 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698