Bug 1750525 Comment 34 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

These should be all patches here. I followed in most of the cases the following rules:

- ignore dispatch failures when sending `__delete__` messages
- remove `MOZ_CRASH_UNLESS_FUZZING` where we already return IPC_FAIL
- give the remaining IPC_FAIL a reason

In some cases there still might be more IPC_FAIL cases than strictly necessary, but if they do not bite us, that's probably ok and we do not need to spend too much time reasoning about them.

The remaining `MOZ_CRASH_UNLESS_FUZZING` should be handled in bug 1751371 as they are not directly IPC_FAIL related.

FWIW, here comes [a try run on the entire patch stack](https://treeherder.mozilla.org/#/jobs?repo=try&revision=0b1f5b1c1ffe58ddc8d2d18fb4372cefa1b46618).
These should be all patches here. I followed in most of the cases the following rules:

- ignore dispatch failures when sending `__delete__` messages
- remove `MOZ_CRASH_UNLESS_FUZZING` where we already return IPC_FAIL (given that this triggers already a `MOZ_ASSERT_UNLESS_FUZZING`)
- give the remaining IPC_FAIL a reason

In some cases there still might be more IPC_FAIL cases than strictly necessary, but if they do not bite us, that's probably ok and we do not need to spend too much time reasoning about them.

The remaining `MOZ_CRASH_UNLESS_FUZZING` should be handled in bug 1751371 as they are not directly IPC_FAIL related.

FWIW, here comes [a try run on the entire patch stack](https://treeherder.mozilla.org/#/jobs?repo=try&revision=0b1f5b1c1ffe58ddc8d2d18fb4372cefa1b46618).

Back to Bug 1750525 Comment 34