It seems unnecessary to call `tostring(definition_xml)` and then immediately parse it again in `_parse_video_xml`. We could change `_parse_video_xml` to expect an object, rather than a string. It's only being called from two places, both in this file.