Commit Graph

274 Commits

Author SHA1 Message Date
Nathan Froyd
99c976cb15 Bug 1298676 - part 3 - disable I/O interposition when manually taking crashdumps; r=ted.mielczarek 2017-01-10 16:31:25 -05:00
Markus Stange
a24593a89f Bug 1323100 - Assign names to all remaining threads that are created through NS_NewThread and create them using NS_NewNamedThread instead. r=froydnj
MozReview-Commit-ID: 7W1dt2BBKJZ
2016-12-21 11:43:50 +01:00
Gabriele Svelto
921d52f071 Bug 1293656 - Send crash pings for content crashes complete with stack traces r=bsmedberg
* * *
Bug 1293656 - Fix the test_process_error.xul test
2016-10-19 12:51:29 +02:00
Gabriele Svelto
7af207f5bf Bug 1293656 - Store all crash annotations in the crash manager's database and filter out privacy-sensitive ones when sending a crash ping r=bsmedberg 2016-10-19 12:48:19 +02:00
Gabriele Svelto
1fc9fa978f Bug 1293656 - Clean up the exception handler code r=ted 2016-10-19 12:38:56 +02:00
Tomislav Jurin
9ff3007b90 Bug 1296189 - Replace NS_RUNTIMEABORT("some string literal message") with MOZ_CRASH(). r=froydnj 2016-12-02 13:46:53 -08:00
Nathan Froyd
321e839727 Bug 1318739 - remove DoFindInReadable from nsExceptionHandler; r=ted.mielczarek
We now have an overload for the simple case of finding |key| in some
|string|, so we don't need this utility function anymore, which
presumably dates from when said overload did not exist or was
inaccessible due to library visibility, or something.
2016-11-28 09:33:07 -04:00
Randall Barker
c82d8aa286 Bug 1314466 - part 2, Allow android to set channel file descriptor for crash reporter r=froydnj 2016-11-14 16:45:23 -08:00
David Major
f0780703de Bug 1316168: Fix the clang-cl build in nsExceptionHandler.cpp. r=froydnj
MozReview-Commit-ID: HdUp3dGRmcq
2016-11-09 10:39:20 -06:00
Emanuel Hoogeveen
cdea0978c6 Bug 1309573 - Part 1: Define the crash reason in MFBT to let everything use it. r=froydnj 2016-11-06 22:57:00 -05:00
Ting-Yu Chou
8f9961e3fc Bug 1301022 - Add an API for annotating pending IPC messages. r=ted
MozReview-Commit-ID: GiAMUnjYjvI
2016-09-20 14:10:43 +08:00
David Anderson
c803f94cf6 Add nsExceptionHandler support for CrashReporterClient. (bug 1278717 part 2, r=billm) 2016-10-11 14:25:21 -07:00
Phil Ringnalda
678c8ee447 Merge m-i to m-c, a=merge
MozReview-Commit-ID: 93ZdJbK1x05
2016-10-06 19:58:18 -07:00
Gian-Carlo Pascutto
3262985f94 Bug 1289718 - Clean up stat/stat64 wrapper. Deal with non-default TMPDIR. r=jld
MozReview-Commit-ID: DW63be9qd3z
2016-10-06 13:25:13 +02:00
Gabriele Svelto
088fc29892 Bug 1264367 - Upgrade breakpad to upstream revision 704f41ec901c419f8c321742114b415e6f5ceacc r=ted
* * *
Bug 1264367 - Accomodate for the new version of Breakpad
2016-10-05 11:46:07 +02:00
Ting-Yu Chou
5455e4f1ca Bug 1301022 - Back out crash annotations which doesn't work in child process. r=me
MozReview-Commit-ID: BMq1lxwDO2u
2016-10-03 10:08:11 +08:00
Wes Kocher
d31d166f3f Backed out changeset 1adfbab318a6 (bug 1264367) for breaking crash stacks CLOSED TREE 2016-10-03 14:50:27 -07:00
Gabriele Svelto
8bd242ea95 Bug 1264367 - Upgrade breakpad to upstream revision 704f41ec901c419f8c321742114b415e6f5ceacc r=ted 2016-10-03 13:43:28 +02:00
Nathan Froyd
3362bded10 Bug 1305422 - part 16 - don't call size_backward in nsExceptionHandler.cpp; r=ted.mielczarek 2016-09-29 22:33:57 -04:00
Phil Ringnalda
9f04a60b90 Backed out changeset ae50c61cb134 (bug 1264367) for breaking crash stacks on at least Android and Linux 2016-09-21 13:26:24 -07:00
Gabriele Svelto
53ad14981b Bug 1264367 - Upgrade breakpad to upstream revision 704f41ec901c419f8c321742114b415e6f5ceacc r=ted 2016-09-21 17:44:50 +02:00
Ting-Yu Chou
6c461e7e7e Bug 1301022 - Add an API for annotating pending IPC messages. r=ted
MozReview-Commit-ID: GiAMUnjYjvI
2016-09-20 14:10:43 +08:00
Phil Ringnalda
40ceb1671e Backed out changeset af84ff6bfd7e (bug 1264367) for causing crash stacks to lack symbols
CLOSED TREE
2016-09-15 19:15:45 -07:00
Gabriele Svelto
ae9c4a5ed1 Bug 1264367 - Upgrade breakpad to upstream revision 704f41ec901c419f8c321742114b415e6f5ceacc r=ted 2016-09-15 17:31:13 +02:00
Sylvestre Ledru
2c97162008 Bug 1298701 - Separate the two strings (StartupCrash & TelemetryEnvironment) in the kCrashEventAnnotations structure r=njn
MozReview-Commit-ID: Cig5cGWnwqG
2016-08-28 21:54:48 +02:00
Nicholas Nethercote
5f8bae5b6a Bug 1295934 - Add a "StartupCrash" crash report annotation. r=ted. 2016-08-25 11:16:49 +10:00
Kan-Ru Chen
a9b19d0584 Bug 1297276 - Rename mfbt/unused.h to mfbt/Unused.h for consistency. r=froydnj
The patch is generated from following command:

  rgrep -l unused.h|xargs sed -i -e s,mozilla/unused.h,mozilla/Unused.h,

MozReview-Commit-ID: AtLcWApZfES
2016-08-24 14:47:04 +08:00
Igor
9c81c3c1ee Bug 1293384 - Part 2: Rename Snprintf.h header to Sprintf.h. r=froydnj 2016-08-14 23:43:21 -07:00
Igor
972b8460e2 Bug 1293384 - Part 1: Rename snprintf_literal to SprintfLiteral. r=froydnj 2016-08-14 23:44:00 -07:00
Nicholas Nethercote
e1817288cf Bug 1293117 (part 5) - Change many NS_METHOD occurrences to NS_IMETHOD. r=froydnj.
This patch makes the following changes on many in-class methods.

- NS_METHOD F() override;      --> NS_IMETHOD F() override;
- NS_METHOD F() override {...} --> NS_IMETHOD F() override {...}
- NS_METHOD F() final;         --> NS_IMETHOD F() final;
- NS_METHOD F() final {...}    --> NS_IMETHOD F() final {...}

Using NS_IMETHOD is the preferred way of marking in-class virtual methods.
Although these transformations add an explicit |virtual|, they are safe --
there's an implicit |virtual| anyway because |override| and |final| only work
with virtual methods.
2016-08-08 10:54:50 +10:00
Nicholas Nethercote
1f65390cc9 Bug 1293603 (part 2) - Make Run() declarations consistent. r=erahm.
This patch makes most Run() declarations in subclasses of nsIRunnable have the
same form: |NS_IMETHOD Run() override|.

As a result of these changes, I had to add |override| to a couple of other
functions to satisfy clang's -Winconsistent-missing-override warning.
2016-08-08 12:18:10 +10:00
Chris Peterson
0eccc785ff Bug 1277106 - Part 1: Use VS2015's real char16_t instead of aliasing wchar_t. r=Waldo
and remove MOZ_CHAR16_IS_NOT_WCHAR #ifdefs.
2016-07-20 22:01:43 -07:00
Ting-Yu Chou
b3957781cb Bug 1267329 - part 2 - Record the memory information for the minidumps on Windows. r=ted 2016-06-30 08:59:02 +08:00
Kirk Steuber
2739fab23f Bug 1263774 - Include memory reports in content process crash reports. r=mccr8, r=aklotz
MozReview-Commit-ID: 7y3GFBZxjsS
2016-06-21 11:45:25 -07:00
Dylan Roeh
e643c94b11 Bug 1262536 - Report minidump success/failure in CrashReporter. r=snorp 2016-05-17 15:53:15 -05:00
Milan Sreckovic
f0d03b465f Bug 1250637: Allow child process crash annotation off main thread. r=aklotz
MozReview-Commit-ID: B1oq7KlZrff
2016-05-10 15:59:00 -04:00
Kyle Huey
a9cf047227 Bug 1265927: Move nsRunnable to mozilla::Runnable, CancelableRunnable to mozilla::CancelableRunnable. r=froydnj 2016-04-25 17:23:21 -07:00
Ted Mielczarek
aa18b7bab2 bug 1264242 - Write memory info streams for Windows Minidumps of child processes. r=bsmedberg
MozReview-Commit-ID: COHFdNiJIqm
2016-04-13 08:05:46 -04:00
Cervantes Yu
f185bfcd3f Bug 1257486 - Annotate global memory status in the crash reporter for child process; r=bsmedberg
MozReview-Commit-ID: ExVZS3aAp41
2016-03-29 19:16:09 +08:00
Aaron Klotz
c8a0d9ab7b Bug 1256541: Fix incorrect generation of path for child process .extra files when content sandboxing is disabled; r=bsmedberg
MozReview-Commit-ID: JQgqlntQ6cu
2016-03-16 12:35:50 -06:00
Wes Kocher
090bbab1db Backed out changeset 0827e0e5d9d4 (bug 1256541) for timeouts and crashes in browser_CTP_crashreporting.js
MozReview-Commit-ID: 704JC6wknXc
2016-03-24 13:04:48 -07:00
Aaron Klotz
b568d00e85 Bug 1256541: Fix incorrect generation of path for child process .extra files when content sandboxing is disabled; r=bsmedberg
Since the minidump path can be overridden programmatically in the chrome
process, using that path as the base for .extra files won't work since content
is unaware of it.

This patch changes everything to use the temp path when MOZ_CONTENT_SANDBOX is
not defined or when sandboxing is disabled via pref. It also moves the derivation
of the content temp path out of exception context on Windows and Mac, as I
found out that those functions touch the heap.

I also noticed that xpcshell is not sandbox-aware when utilized as a parent
process. I've filed bug 1257098 to take care of that, but this patch includes a
hack for the immediate term.

MozReview-Commit-ID: 3SIB5Nihqxh
2016-03-16 12:35:50 -06:00
Benjamin Smedberg
f5c327d5c6 Bug 1257209 - record uptime in crash metadata/pings, r=gfritzsche
MozReview-Commit-ID: 8MVRwALXvW5
2016-03-16 16:59:06 -04:00
Benjamin Smedberg
8bdab4404f Bug 1187270 - Add Telemetry session ID to crash annotations, r=gfritzsche
MozReview-Commit-ID: GwVI0dfsT4H
2016-03-14 09:31:19 -04:00
Daniel Holbert
ca38f91a55 backout d85b18faba72 (bug 1187270) for build bustage. 2016-03-15 15:51:56 -07:00
Benjamin Smedberg
b97feb5d63 Bug 1187270 - Add Telemetry session ID to crash annotations, r=gfritzsche
MozReview-Commit-ID: GwVI0dfsT4H
2016-03-14 09:31:19 -04:00
Masatoshi Kimura
4afc042f1e Bug 1254963 - Fix build error C2664 on MSVC 2015. r=aklotz 2016-03-11 07:25:33 +09:00
Sebastian Hengst
39bac05fe3 Backed out changeset c1dbec35e0c5 (bug 1254963) for bustage. r=bustage on a CLOSED TREE 2016-03-10 00:14:49 +01:00
Masatoshi Kimura
d97317e861 Bug 1254963 - Fix build error C2664 on MSVC 2015. r=aklotz 2016-03-10 07:45:49 +09:00
Aaron Klotz
558b250488 Bug 1236108: Add support for exception-context annotations for content processes to the crash reporter; r=bsmedberg
This patch redefines XP_PATH_MAX on Windows to be MAX_PATH + 1. I did this
because the longer definition would actually not work with most Windows APIs.
Some APIs can work with longer lengths if the path is prefixed with "\\?\", but
that is not guaranteed in general.

See https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx#maxpath

MozReview-Commit-ID: LhT5oM5hlkl
2016-03-07 12:48:58 -08:00