Following on from
LUCENE-7210 (and others), the xml queryparser has different factories, one for creating normal queries and one for creating span queries.
The former is a protected variable so can be used by derived classes, the latter isn't.
This makes the spanFactory a variable that can be used more easily. No functional changes.