Remove code duplication in FieldsVisitor. elastic#18218 …
It currently has twice the same method, once with a MapperService instance and once with a DocumentMapper. This commits only keeps the former.