This is an archived snapshot of W3C's public bugzilla bug tracker, decommissioned in April 2019. Please see the home page for more details.

Bug 14572 - <track> Collect space characters and discard them after step 8.
Summary: <track> Collect space characters and discard them after step 8.
Status: CLOSED FIXED
Alias: None
Product: WHATWG
Classification: Unclassified
Component: HTML (show other bugs)
Version: unspecified
Hardware: Other other
: P3 normal
Target Milestone: Unsorted
Assignee: Ian 'Hixie' Hickson
QA Contact: contributor
URL: http://www.whatwg.org/specs/web-apps/...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-27 12:41 UTC by contributor
Modified: 2012-07-18 18:40 UTC (History)
5 users (show)

See Also:


Attachments

Description contributor 2011-10-27 12:41:36 UTC
Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/the-video-element.html
Multipage: http://www.whatwg.org/C#parsing-0
Complete: http://www.whatwg.org/c#parsing-0

Comment:
Collect space characters and discard them after step 8.

Posted from: 2001:4c28:a030:30:72f3:95ff:fe00:4501
User agent: Opera/9.80 (X11; Linux x86_64; U; sv) Presto/2.9.168 Version/11.51
Comment 1 Fredrik S 2011-10-27 12:58:59 UTC
In http://www.whatwg.org/html#parse-the-webvtt-settings, there seems to be a step missing to handle any separating whitespace between settings.

Consider for example the settings string:

"T:75% L:10"

After parsing the "T:75%" part the algorithm is looking at " L:10" and is at step 2. Step 3 will then set 'setting' to " " and move 'position'. Step 5 will then set 'setting' to "" and step 6 consumes the "L". In step 8 the Otherwise-branch is taken and ":10" is consumed, resulting in the loss of the L-setting.
Comment 2 Ian 'Hixie' Hickson 2011-10-30 17:57:33 UTC
EDITOR'S RESPONSE: This is an Editor's Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document:
   http://dev.w3.org/html5/decision-policy/decision-policy.html

Status: Accepted
Change Description: see diff given below
Rationale: Concurred with reporter's comments.
Comment 3 contributor 2011-10-30 17:58:11 UTC
Checked in as WHATWG revision r6788.
Check-in comment: Move a skip whitespace step to where it belongs.
http://html5.org/tools/web-apps-tracker?from=6787&to=6788
Comment 4 Fredrik S 2011-10-31 07:55:18 UTC
The change would be ok if the label "Settings" was moved to the same step.
Comment 5 contributor 2011-11-01 06:09:52 UTC
Checked in as WHATWG revision r6801.
Check-in comment: fix incomplete earlier edit in r6788
http://html5.org/tools/web-apps-tracker?from=6800&to=6801