`dom.ts#getFocusedWindow()` has a weird redundant check

[bpasero]

https://github.com/microsoft/vscode/blob/496ebc4723371f29c9ffa0319dcccb2d7bee7ee0/src/vs/base/browser/dom.ts#L945-L952

Unless I am missing something: document.defaultView?.window.document === document. So whats the idea of using document.defaultView?.window in that method?

[meganrogge]

Here's what I see when I make that substitution

[bpasero]

@meganrogge I see, a suggested fix in https://github.com/microsoft/vscode/pull/249511

Next time, please feel free to just reach out to me to ask for advice, that lint rule is obviously not catching that scenario well.