1896636 - Remove the crash reporter injector code

Status: RESOLVED FIXED

(Toolkit :: Crash Reporting, task)

Description

[Gabriele Svelto [:gsvelto]]

The crash reporter injector code was used to inject Breakpad into Flash processes. It's not used anymore and hasn't been for a while. The closest thing we have to it is used by SpiderMonkey but doesn't rely on the in-tree injector (see bug 1522180 comment 3) and we hope to replace it with the OOP generator.

Comment 1

[Gabriele Svelto [:gsvelto]]

Attachment: Bug 1896636 - Remove the crash reporter injector r=KrisWright

This was used to inject our crash reporting infrastructure into the Flash
processes and has been unused since Flash support has been removed.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8f84d1a78c5c
Remove the crash reporter injector r=KrisWright

Comment 3

[Cristian Tuns]

Backed out for causing multiple failures

• Backout link

• Push with failures

• Failure Log 1

• Failure line: TEST-UNEXPECTED-FAIL | toolkit/crashreporter/test/unit_ipc/test_content_annotation.js | xpcshell return code: 0

• Failure Log 2

• Failure line: PROCESS-CRASH | application crashed [@ js::ctypes::ConvertToJS] | xpcshell-remote.toml:toolkit/components/extensions/test/xpcshell/test_ext_background_early_shutdown.js

• Failure Log 3

• Failure line: TEST-UNEXPECTED-FAIL | toolkit/crashreporter/test/browser/browser_cpu_microcode.js | contains CPU microcode version -

Also: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=8f84d1a78c5cae71b879a33a7aa8528fdf468f7b&searchStr=gv-junit-fis&selectedTaskRun=Wq-pnGlhTbuzLnQ0h5Lhjw.0

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c37af59f3ad6
Remove the crash reporter injector r=KrisWright

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/c37af59f3ad6