hmm. I'm not the original author here (is it @dmitchell or @peter-fogg?), so I'm repairing existing implementation. There's a turnaround call to _parse_video_xml() - which AFAIK - is to provide a translation from XML formats to the the new entity model.

But, that said, I think I see what you mean about video.data should have everything in there already.