mozilla / vtt.js

A JavaScript implementation of the WebVTT specification
http://dev.w3.org/html5/webvtt/
Apache License 2.0
484 stars 94 forks source link

Parsing of 60:00.000 is wrong #345

Open zcorpan opened 8 years ago

zcorpan commented 8 years ago

https://github.com/mozilla/vtt.js/blob/master/lib/vtt.js#L73..L75

The spec requires that 60:00.000 should return an error, not parse as [hours]:[minutes].[milliseconds], AFAICT.

https://w3c.github.io/webvtt/#collect-a-webvtt-timestamp step 12

12 If most significant units is hours, or [...], run these substeps:

  1. If [...] the character at position is not a U+003A COLON character (:), then return an error and abort these steps.