Describe the PR

• Remove XML parser options, which now use their previous defaults
• Remove the abstract XML visitor rule. It's not used, and it looks unnecessary. Why would one use a java visitor over XPath for an xml rule... Anyway if we want to support that in pmd 7, we should use the new AstVisitor interface, and I just don't see the point for now.

Existing parser tests for xml are converted to text comparison tests.

Related issues

• Related to [core] Remove ParserOptions #2602

Ready?

• Deprecate AbstractWsdlRule, AbstractDomXmlRule on master
• Added unit tests for fixed bug/feature
• Passing all unit tests
• Complete build ./mvnw clean verify passes (checked automatically by travis)
• Added (in-code) documentation (if needed)