934288 - Add checks to make sure the hidden window doesn't contain anything unexpected when browser is shutdown

Status: NEW

(Toolkit :: General, defect)

Description

[Olli Pettay [:smaug][[e-mail]]]

We should have some checks to ensure that whoever has used inner window also removes
the stuff from there. Inner window's document during shutdown should look like it looks
initially and window scope shouldn't have anything special either.

Comment 1

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Surely inner means hidden here ...?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Yes, silly me.

Comment 3

[Nicholas Nethercote [inactive]]

Smaug, how difficult is this?  It doesn't sound too hard.

AIUI, this would overlap somewhat with the checking in bug 932898, but not entirely.

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

(In reply to Nicholas Nethercote [:njn] from comment #3)
> AIUI, this would overlap somewhat with the checking in bug 932898, but not
> entirely.

Yeah, any DOMWindows and DocShells left in the hiddenWindow would be detected by it.

Comment 5

[Olli Pettay [:smaug][[e-mail]]]

Some rather dummy check should be easy. Effectively store
document.documentElement.innerHTML somewhere and compare it to document.documentElement.innerHTML
when we're about to close and do something similar to properties in the window scope.