964625 - The WebVTT parser does not require whitespace between timings and settings

Status: VERIFIED FIXED

(Core :: General, defect)

Description

[Philip Jägenstedt [:foolip]]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/32.0.1700.76 Safari/537.36 OPR/19.0.1326.45 (Edition Next)

Steps to reproduce:

Feed this to the WebVTT parser:

WEBVTT

00:00.000 --> 00:01.000line:40% size:50%
cue text



Actual results:

A cue with line "auto" and size 50 is added.


Expected results:

A cue with line 40 and size 50 should be added.

Please see the discussion in https://www.w3.org/Bugs/Public/show_bug.cgi?id=22104 and object there if you have deliberately ignored the spec on this point and really want it to change.

Comment 1

[Rick Eyre (:reyre)]

Upstream change landed in vtt.js https://github.com/mozilla/vtt.js/pull/227.

Comment 2

[Rick Eyre (:reyre)]

Upstream changes landed in bug 965246.

Comment 3

[Ada [:adalucinet]]

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0
Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:29.0) Gecko/20100101 Firefox/29.0

Reproduced with Nightly from 2014-01-20: a cue with line "auto" is added instead of 40.
Verified as fixed with Fx 29 beta 4 (Build ID: 20140331125246) on Windows 7 x64, Mac OS X 10.9 and Ubuntu 13.10 x64 with a HMTL page which contains:
<video src="http://people.mozilla.org/~rgiles/2013/demo.webm" controls >
   <track src="sample.vtt" kind="subtitles" srclang="en" label="English" />
</video>
and the corresponding sample.vtt file saved in the same folder.