1541891 - Make BrowsingContext::GetParent ~= GetSameTypeParent(), which respects MozBrowser boundaries.

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, task, P2)

Comment 1

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

For when you get back, I was under the impression that this already works correctly. Are there known discrepancies here?

Comment 2

[Andreas Farre [:farre]]

Yeah, so I'm pretty certain that this should hold. Let me run some tests on try to see what happens if we make BrowsingContext::GetParent === GetSameTypeParent()

Comment 3

[Andreas Farre [:farre]]

So I think that I filed this on your feedback for something. It should be correct as is, but I'm running a try with asserts, if that is horribly orange I'll reconsider otherwise this is just a wontfix:

Comment 4

[Andreas Farre [:farre]]

So I think that I filed this on your feedback for something. It should be correct as is, but I'm running a try with asserts, if that is horribly orange I'll reconsider otherwise this is just a wontfix:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=afe096b3018881776bfd5bf7c1c72b2fa2109ba7

Comment 5

[Andreas Farre [:farre]]

So the try from Comment 4 was pretty orange, so I'm running a test that asserts the other way around:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=ead6d4a7b71e8a545bb6e7f69215ee69ad441748

because I think that at GetSameTypeParent might be the same as GetParent at the time of GetSameTypeParent being called, but not the other way around. This would then also be because of how docshells are destroyed and how that differs from how browsing contexts being detached.

Comment 6

[Andreas Farre [:farre]]

So this seems equally bad. Will dig into it to see exactly what's happening in the latter.

Comment 7

[Andreas Farre [:farre]]

So, checking that when we do get a docshell from GetSameTypeParen, then that condition holds for the browsing contexts as well: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ec672943b4c789c81c22f3ff2637c9ea1576eb0f

Comment 8

[Andreas Farre [:farre]]

Yep, the last of the runs seems fine. The former two is the old issue with tearing down browsing contexts isn't the same as tearing down docshells.