Assertion failure: browsingContext->IsOwnedByProcess(childId), at dom/ipc/WindowGlobalParent.cpp:328
Categories
(Core :: DOM: Content Processes, defect, P2)
Tracking
()
Fission Milestone | M5 |
People
(Reporter: hiro, Assigned: jya)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 obsolete file)
I was bitten by this assertion when I was writing a mochitest for bug 1553012. And it seems that this assertion commonly happens when setting src attribute for iframe.
You can see the assertion by running ./mach test --setpref "fission.autostart=true" dom/base/test/test_bug590870.html
.
Reporter | ||
Updated•5 years ago
|
Comment 2•5 years ago
|
||
The priority flag is not set for this bug.
:jimm, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•5 years ago
|
Assignee | ||
Comment 3•5 years ago
|
||
While ideally we should prevent this condition from ever happening, we can simply reject the promise for now which will abort the pending SessionStore restore.
In the short future, the most common cause for this race will be fixed by not requiring a process switch for each redirect
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 4•5 years ago
|
||
Roll some unfixed bugs from Fission Milestone M4 to M5
0ee3c76a-bc79-4eb2-8d12-05dc0b68e732
Comment 5•5 years ago
|
||
kmag thinks he fixed this problem. The test_bug590870.html test is not currently skipped on Fission.
@ Matt, can you still reproduce this crash on cnn.com?
Assignee | ||
Comment 6•5 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #5)
kmag thinks he fixed this problem. The test_bug590870.html test is not currently skipped on Fission.
@ Matt, can you still reproduce this crash on cnn.com?
I was the one who worked on that bug and could reproduce it easily.
The condition of the bug can no longer happen with DocumentChannel as we no longer have the possibility of a race between multiple processes being spawned for a single redirects.
Assignee | ||
Updated•5 years ago
|
Updated•3 years ago
|
Description
•