Closed
Bug 964625
Opened 11 years ago
Closed 11 years ago
The WebVTT parser does not require whitespace between timings and settings
Categories
(Core :: General, defect)
Tracking
()
VERIFIED
FIXED
mozilla29
People
(Reporter: foolip, Assigned: reyre)
References
Details
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.
Assignee | ||
Updated•11 years ago
|
QA Contact: rick.eyre
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rick.eyre
QA Contact: rick.eyre
Assignee | ||
Comment 1•11 years ago
|
||
Upstream change landed in vtt.js https://github.com/mozilla/vtt.js/pull/227.
Assignee | ||
Comment 2•11 years ago
|
||
Upstream changes landed in bug 965246.
Status: UNCONFIRMED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → mozilla29
Comment 3•11 years ago
|
||
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.
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•