18590 2012-08-16 12:54:56 +0000 Firing an event is sometimes missing a target and "queue a task" 2012-10-12 12:06:32 +0000 1 1 1 Unclassified HTML WG Media Source Extensions unspecified PC Linux VERIFIED FIXED P2 normal --- 1 philipj adrianba acolwell mike public-html-media public-html-bugzilla oldest_to_newest 72277 0 philipj 2012-08-16 12:54:56 +0000 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. 75618 1 acolwell 2012-10-08 19:52:36 +0000 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 76090 2 philipj 2012-10-12 12:06:32 +0000 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.