I think the code would be clearer if this was ReadFromXML() or LoadXML() or similar.