Open
Bug 1318196
Opened 8 years ago
Updated 1 year ago
StableState should be fired before second <script> tag and right after the </video>.
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
NEW
Tracking | Status | |
---|---|---|
firefox53 | --- | affected |
People
(Reporter: ctai, Unassigned)
References
(Depends on 1 open bug, Blocks 4 open bugs)
Details
Per talk with smaug, [1] and [2] should be valid test cases. [1]: view-source:http://w3c-test.org/html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-source-media.html [2]: view-source:http://w3c-test.org/html/semantics/embedded-content/media-elements/loading-the-media-resource/resource-selection-candidate-insert-before.html
Reporter | ||
Comment 1•8 years ago
|
||
once this bug fixed, below task can be enabled. resource-selection-candidate-insert-before.html resource-selection-candidate-moved.html resource-selection-candidate-remove-addEventListener.html resource-selection-candidate-remove-onerror.html resource-selection-invoke-insert-source-networkState.html resource-selection-invoke-in-sync-event.html resource-selection-invoke-load.html resource-selection-invoke-pause.html resource-selection-invoke-play.html resource-selection-remove-source.html resource-selection-remove-src.html
Reporter | ||
Comment 12•8 years ago
|
||
Also resource-selection-pointer-insert-source.html, resource-selection-pointer-remove-source.html resource-selection-source-media.html Those are related to </video> tag.
Reporter | ||
Comment 13•8 years ago
|
||
(In reply to Chia-hung Tai [:ctai :ctai_mozilla :cht] from comment #1) > once this bug fixed, below task can be enabled. > > resource-selection-candidate-insert-before.html > resource-selection-candidate-moved.html > resource-selection-candidate-remove-addEventListener.html > resource-selection-candidate-remove-onerror.html > resource-selection-invoke-insert-source-networkState.html > resource-selection-invoke-in-sync-event.html > resource-selection-invoke-load.html > resource-selection-invoke-pause.html > resource-selection-invoke-play.html > resource-selection-remove-source.html > resource-selection-remove-src.html Those are related to two script case.
Reporter | ||
Comment 14•8 years ago
|
||
Hi, Smaug, Per talk in email. Above test cases are related to microtask checkpoint. Those test cases should be passed after this bug fixed. Please confirm it on bugzilla. :)
Flags: needinfo?(bugs)
Comment 15•8 years ago
|
||
We do have microtasks in right place but our stable state doesn't follow microtask scheduling. Filed bug 1318316
Depends on: 1318316
Flags: needinfo?(bugs)
Updated•8 years ago
|
Summary: Microtask checkpoint should be fired before second <script> tag and right after the </video>. → StableState should be fired before second <script> tag and right after the </video>.
Comment 16•8 years ago
|
||
Chia-hung, are you able to work on this? I marked this P1 due to the high number of intermittents this blocks.
Flags: needinfo?(ctai)
Priority: -- → P1
Comment 17•8 years ago
|
||
This is basically a dup of bug 1318316, which depends on the promise+microtask scheduling stuff I've been doing.
Reporter | ||
Comment 18•8 years ago
|
||
According to Smaug's comment, that should be done by smaug.
Flags: needinfo?(ctai)
Comment 20•8 years ago
|
||
Doesn't really matter. Currently this nicely covers the cases where StableState handling affects to video handling (kind of a meta bug for that) and bug 1318316 then covers the actual change. up to you.
Flags: needinfo?(bugs)
Comment 21•8 years ago
|
||
Is there any spec. to say each <script> should run on its own macro task so the micro tasks schedules by the 1st <script> should happen before the 2nd <script>?
Comment 22•7 years ago
|
||
There are no macro tasks ;) But HTML spec defines that microtask checkpoint happens before <script> execution.
Comment 35•7 years ago
|
||
This doesn't sound something we plan to ship in this release, re-set the priority.
Priority: P1 → P3
Assignee | ||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•