1563604 - Detect invariant-violating sends of BrowsingContext over IPC earlier

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, task, P2)

Comment 1

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1563604 - Part 1: Confirm detach calls before dropping refs,

This ensures that the object is not discarded while there are still in-flight
messages from the parent process, which could cause a content process crash if a
CC was run before any pending messages have arrived.

Comment 2

[Nika Layzell [:nika] (ni? for response)]

Attachment: Bug 1563604 - Part 2: Assert on the sending side that BrowsingContexts are in valid states,

This should help us more easily identify where the crashing issues are coming
from. The issue in part 1 was found due to these assertion changes.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/360b12c6b3e4
Part 1: Confirm detach calls before dropping refs, r=peterv
https://hg.mozilla.org/integration/autoland/rev/54d9810b02ae
Part 2: Assert on the sending side that BrowsingContexts are in valid states, r=peterv

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/360b12c6b3e4
https://hg.mozilla.org/mozilla-central/rev/54d9810b02ae