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 18590 - Firing an event is sometimes missing a target and "queue a task"
Summary: Firing an event is sometimes missing a target and "queue a task"
Status: VERIFIED FIXED
Alias: None
Product: HTML WG
Classification: Unclassified
Component: Media Source Extensions (show other bugs)
Version: unspecified
Hardware: PC Linux
: P2 normal
Target Milestone: ---
Assignee: Adrian Bateman [MSFT]
QA Contact: HTML WG Bugzilla archive list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-16 12:54 UTC by Philip Jägenstedt
Modified: 2012-10-12 12:06 UTC (History)
3 users (show)

See Also:


Attachments

Description Philip Jägenstedt 2012-08-16 12:54:56 UTC
In HTML, the phrasing is "fire a simple event named X at Y", sometimes preceded by "queue a task to".

There are several instances where the target or "queue a task" is missing in MSE:

http://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html#mediasource-attach

"Fire a simple event named sourceopen." doesn't name the target of the event

http://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html#mediasource-detach

"fire an emptied event" is inconsistent in phrasing and also missing the target.

http://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html#dom-duration

"Update HTMLMediaElement.duration to the new duration and schedule the appropriate durationchange event to fire" is using "schedule" instead of "queue a task", but presumably means the same thing? (Note also that invoking "When the length of the media resource changes to a known value" in HTML by phrasing this differently would remove the need to mention the event at all.)

http://dvcs.w3.org/hg/html-media/raw-file/tip/media-source/media-source.html#dom-addsourcebuffer

"Add the new object to sourceBuffers and fire a addsourcebuffer on that object." This is the really problematic instance, the one I filed this bug for. Since this is inside addSourceBuffer, failing to say "queue a task" means that the event should be fired synchronously before the method returns. This is almost certainly not intended.

This is not a full list, search for "event" for more.
Comment 1 Aaron Colwell (c) 2012-10-08 19:52:36 UTC
Changes committed:
http://dvcs.w3.org/hg/html-media/rev/c3de559a1c37

There are still a few instances in Section 2, but I expect those to go away when I start addressing https://www.w3.org/Bugs/Public/show_bug.cgi?id=18575
Comment 2 Philip Jägenstedt 2012-10-12 12:06:32 UTC
I've verified that the "queue a task to" and the event targets are everywhere except in section 2, although I haven't carefully reviewed that all of the event targets are correct.

Editorial: "queue a task to" now often appears at the beginning of a sentence without being capitalized.