Use an nsRefPtr for mTrackElement and cycle collect it.

Maybe the raw pointer was meant to be a weak reference?

Also move cue setting defaults into a convenience method,
call it from both constructors and add missing initializers.

Link to bug 868509 for WebVTTCue stuff.
Document bugs for XXX comments.