fix(checks/has-visible-text): work with virtual and serial nodes

[straker]

I noticed that named-from-contents had an exception for table elements to always be considered named from contents. The comment said it was because the implicit role was too over-assertive but I couldn't find any tests in the named-from-contents test suite that showed an example nor could I find a reason in the original pr it was created.

So I removed the exception and just allowed table role to be named from contents. This seemed like a better place for it and still allowed all the tests to pass (without the exception two of our accessible name acceptance tests failed as they required the table element to be named from contents).

Part of issue: #2183

Reviewer checks

Required fields, to be filled out by PR reviewer(s)

• Follows the commit message policy, appropriate for next version
• Code is reviewed for security