[Bug] Conflicting line of code in src/dom.js causing infinte loop. (waitForDocumentBody) about belter

[bluepnume]

I agree the throw is unreachable. That may just be an artifact of making Flow happy (since document.body is always an optional in Flow, throwing is the only way to make Flow happy that we are returning a definite HTMLBodyElement type rather than a ?HTMLBodyElement).

That said; I don't think we can change the behavior of waitForDocumentReady or isDocumentReady at this point, to not depend on document.body being present. That can be changed in a new major version of belter -- but if we remove that now, that is likely to be breaking behavior.

[Cijin]

Yes, and to avoid that I could remove the conditional statement. Because if waitForDocumentReady has resolved that means document.body has to be present. That could make flow happy.

I agree with you on the second part. Changing waitForDocumentReady or isDocumentReady would cause breaking changes.