fix(byRole): Don't assume window context

[eps1lon]

What:

Get window from passed element

Why:

element might be passed from another window/frame

How:

Use element.ownerDocument.defaultView. This is safe to evaluate since we expect an Element. ownerDocument is only null for Document (https://developer.mozilla.org/en-US/docs/Web/API/Node/ownerDocument)

Checklist:

• [ ] Documentation added to the
  docs site
• ~[ ] I've prepared a PR for types targetting ~https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/testing-library__dom
• Tests: To be honest I wouldn't know if these can create different results. As far as I know getComputedStyle never reads from its window (this) but only from the window of the passed element (at least the jsdom implementation). So from my knowledge this is only useful if you run inside node without a global DOM.
• Ready to be merged
  

[kentcdodds]

Despite #377, I think we're good to go forward with this one.

[kentcdodds]

🎉 This PR is included in version 6.7.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀