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 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.
(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.

Back to Bug 1646817 Comment 22