[giow] (1) Prevent a race condition between <video onerror> and <body…

… onload> Fixing https://www.w3.org/Bugs/Public/show_bug.cgi?id=18570 Affected topics: Video and Audio git-svn-id: http://svn.whatwg.org/webapps@7494 340c8d12-0b0e-0410-8428-c7bf67bfef74
whatwg · Oct 31, 2012 · 628b8ad · 628b8ad
1 parent 4f64824
commit 628b8ad
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 12 deletions.
diff --git a/complete.html b/complete.html
@@ -29591,6 +29591,12 @@ <h5 id=loading-the-media-resource><span class=secno>4.8.10.5 </span>Loading the
     source</a> in one of the <a href=#task-queue title="task queue">task
     queues</a>, then remove those tasks.</p>
 
+    <p>If there are any <a href=#concept-task title=concept-task>tasks</a> that were <a href=#queue-a-task title="queue a
+    task">queued</a> by the <a href=#concept-media-load-algorithm title=concept-media-load-algorithm>resource selection
+    algorithm</a> for this same <a href=#media-element>media element</a> from the <a href=#dom-manipulation-task-source>DOM manipulation task
+    source</a> in one of the <a href=#task-queue title="task queue">task queues</a>, then remove those
+    tasks.</p>
+
     <p class=note>Basically, pending events and callbacks for the
     media element are discarded when the media element starts loading
     a new resource.</p>
@@ -29785,9 +29791,8 @@ <h5 id=loading-the-media-resource><span class=secno>4.8.10.5 </span>Loading the
 
       </ol></li>
 
-     <li><p><a href=#queue-a-task>Queue a task</a> to <a href=#fire-a-simple-event>fire a simple
-     event</a> named <code title=event-media-error><a href=#event-media-error>error</a></code>
-     at the <a href=#media-element>media element</a>.</li>
+     <li><p><a href=#queue-a-task>Queue a task</a> to <a href=#fire-a-simple-event>fire a simple event</a> named <code title=event-media-error><a href=#event-media-error>error</a></code> at the <a href=#media-element>media element</a>, using the <a href=#dom-manipulation-task-source>DOM
+     manipulation task source</a>.</li>
 
      <li><p>Set the element's <a href=#delaying-the-load-event-flag>delaying-the-load-event flag</a>
      to false. This stops <a href=#delay-the-load-event title="delay the load event">delaying
@@ -32510,7 +32515,7 @@ <h6 id=audiotracklist-and-videotracklist-objects><span class=secno>4.8.10.10.1 <
   </table><hr><p>The <a href=#task-source>task source</a> for the <a href=#concept-task title=concept-task>tasks</a> listed in this section is the
   <a href=#dom-manipulation-task-source>DOM manipulation task source</a>.</p>
 
-  <!-- v2 should also fire an event when the list changes -->
+  <!-- v2 should also fire an event when the list changes (but note that that should be on the media element event task source) -->
 
   </div>
 

diff --git a/index b/index
@@ -29591,6 +29591,12 @@ interface <dfn id=htmlaudioelement>HTMLAudioElement</dfn> : <a href=#htmlmediael
     source</a> in one of the <a href=#task-queue title="task queue">task
     queues</a>, then remove those tasks.</p>
 
+    <p>If there are any <a href=#concept-task title=concept-task>tasks</a> that were <a href=#queue-a-task title="queue a
+    task">queued</a> by the <a href=#concept-media-load-algorithm title=concept-media-load-algorithm>resource selection
+    algorithm</a> for this same <a href=#media-element>media element</a> from the <a href=#dom-manipulation-task-source>DOM manipulation task
+    source</a> in one of the <a href=#task-queue title="task queue">task queues</a>, then remove those
+    tasks.</p>
+
     <p class=note>Basically, pending events and callbacks for the
     media element are discarded when the media element starts loading
     a new resource.</p>
@@ -29785,9 +29791,8 @@ interface <dfn id=htmlaudioelement>HTMLAudioElement</dfn> : <a href=#htmlmediael
 
       </ol></li>
 
-     <li><p><a href=#queue-a-task>Queue a task</a> to <a href=#fire-a-simple-event>fire a simple
-     event</a> named <code title=event-media-error><a href=#event-media-error>error</a></code>
-     at the <a href=#media-element>media element</a>.</li>
+     <li><p><a href=#queue-a-task>Queue a task</a> to <a href=#fire-a-simple-event>fire a simple event</a> named <code title=event-media-error><a href=#event-media-error>error</a></code> at the <a href=#media-element>media element</a>, using the <a href=#dom-manipulation-task-source>DOM
+     manipulation task source</a>.</li>
 
      <li><p>Set the element's <a href=#delaying-the-load-event-flag>delaying-the-load-event flag</a>
      to false. This stops <a href=#delay-the-load-event title="delay the load event">delaying
@@ -32510,7 +32515,7 @@ interface <dfn id=videotrack>VideoTrack</dfn> {
   </table><hr><p>The <a href=#task-source>task source</a> for the <a href=#concept-task title=concept-task>tasks</a> listed in this section is the
   <a href=#dom-manipulation-task-source>DOM manipulation task source</a>.</p>
 
-  <!-- v2 should also fire an event when the list changes -->
+  <!-- v2 should also fire an event when the list changes (but note that that should be on the media element event task source) -->
 
   </div>
 

diff --git a/source b/source
@@ -32057,6 +32057,12 @@ interface <dfn>HTMLAudioElement</dfn> : <span>HTMLMediaElement</span> {};</pre>
     source</span> in one of the <span title="task queue">task
     queues</span>, then remove those tasks.</p>
 
+    <p>If there are any <span title="concept-task">tasks</span> that were <span title="queue a
+    task">queued</span> by the <span title="concept-media-load-algorithm">resource selection
+    algorithm</span> for this same <span>media element</span> from the <span>DOM manipulation task
+    source</span> in one of the <span title="task queue">task queues</span>, then remove those
+    tasks.</p>
+
     <p class="note">Basically, pending events and callbacks for the
     media element are discarded when the media element starts loading
     a new resource.</p>
@@ -32302,9 +32308,9 @@ interface <dfn>HTMLAudioElement</dfn> : <span>HTMLMediaElement</span> {};</pre>
 
      </li>
 
-     <li><p><span>Queue a task</span> to <span>fire a simple
-     event</span> named <code title="event-media-error">error</code>
-     at the <span>media element</span>.</p></li>
+     <li><p><span>Queue a task</span> to <span>fire a simple event</span> named <code
+     title="event-media-error">error</code> at the <span>media element</span>, using the <span>DOM
+     manipulation task source</span>.</p></li>
 
      <li><p>Set the element's <span>delaying-the-load-event flag</span>
      to false. This stops <span title="delay the load event">delaying
@@ -35549,7 +35555,7 @@ interface <dfn>VideoTrack</dfn> {
   title="concept-task">tasks</span> listed in this section is the
   <span>DOM manipulation task source</span>.</p>
 
-  <!-- v2 should also fire an event when the list changes -->
+  <!-- v2 should also fire an event when the list changes (but note that that should be on the media element event task source) -->
 
   </div>