Create a more performant variant of SimpleAttributeDefinition.parse

The parse method takes a Location as an argument, requiring the caller to ask the XmlStreamReader for the location whether or not it is needed. It's only needed for creating an error message if the parse method finds an invalid value.

Deprecate this method and instead create a variant where we pass in the XmlStreamReader; let the impl ask for the location only if it is needed.