(In reply to Tomislav Jovanovic :zombie from comment #21) > Overall, I don't think this *must* be fixed in M6c. Yes, some iframes (or just redirects?) in extension popups/sidebars will end up in the wrong process, which we don't wanna, but it shouldn't break many things. @ zombie: kmag hypothesizes this was crash was just a "bad build" from a compiler bug. He suggests we re-land the patch as-is and see if it still crashes. kmag agrees that it doesn't need to block M6c. Deferring to Fission M7 Beta. needinfo'ing Nika to file a follow-up bug to block navigation in this case instead of switching processes or abort if code attempts to swap top-level browsing context. We should fail-safe instead of fail-open.
Bug 1646817 Comment 22 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Tomislav Jovanovic :zombie from comment #21) > Overall, I don't think this *must* be fixed in M6c. Yes, some iframes (or just redirects?) in extension popups/sidebars will end up in the wrong process, which we don't wanna, but it shouldn't break many things. @ zombie: kmag hypothesizes this crash was just a "bad build" from a compiler bug. He suggests we re-land the patch as-is and see if it still crashes. kmag agrees that it doesn't need to block M6c. Deferring to Fission M7 Beta. needinfo'ing Nika to file a follow-up bug to block navigation in this case instead of switching processes or abort if code attempts to swap top-level browsing context. We should fail-safe instead of fail-open.