1679440 - Add test ensuring crash reporter works for background tasks

Status: RESOLVED FIXED

(Toolkit :: Background Tasks, task)

Description

[Nick Alexander :nalexander [he/him]]

This is essentially the "background tasks in Gecko" expression of Bug 1656003. That ticket started a discussion of how to collect crash reports from background tasks that are not part of Gecko/Firefox. To service updates in the background, we're not pursuing that approach; we're going to leverage the existing update service within Gecko. This ticket tracks ensuring that the existing crash reporter works with this new background task mode.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1679440 - Add test ensuring crash reporter works for background tasks. r?Mossop!,gsvelto!

This adds a test to ensure that crash reporting works inside of
application --backgroundtask ... invocations. A test-only crash
task is added that uses CrashTestUtils.jsm to trigger a crash. The
xpcshell harness invokes that background task and processes any
minidump and extras just as it does for existing xpcshell
subprocesses.

The test is homed in toolkit/crashreporter, rather than in
toolkit/components/backgroundtasks, because there is special
handling for CrashTestUtils.jsm and the testcrasher library. It's
probably possible to make that infrastructure usable from multiple
locations but it seems low value.

Depends on D98095

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/310d2116faf7
Add test ensuring crash reporter works for background tasks. r=mossop,gsvelto

Comment 3

[Natalia Csoregi [:nataliaCs]]

Backed out along with Bug 1667276 - https://bugzilla.mozilla.org/show_bug.cgi?id=1667276#c18

Comment 4

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/06ee1ff0214b
Backed out 8 changesets (bug 1679440, bug 1682069, bug 1667276) for causing failures on browser_all_files_referenced.js. CLOSED TREE

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4d325f68ea24
Add test ensuring crash reporter works for background tasks. r=mossop,gsvelto

Comment 6

[Atila Butkovits]

Backed out for causing failure on test_crash_backgroundtask_moz_crash.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/e379907d0452a34155518062e838b465f5e4a37c

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&searchStr=linux%2C18.04%2Cx64%2Copt%2Cxpcshell%2Ctests%2Ctest-linux1804-64%2Fopt-xpcshell-e10s%2Cx2&revision=f06504e3219f5a31ce538829376eeb53be317715&selectedTaskRun=SF47dsaJR4uEMyYymhI9nQ.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=328010578&repo=autoland&lineNumber=3050

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/073d44e79773
Add test ensuring crash reporter works for background tasks. r=mossop,gsvelto

Comment 8

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/073d44e79773