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)

defect

Tracking

()

Tracking Status
firefox53 --- affected

People

(Reporter: ctai, Unassigned)

References

(Depends on 1 open bug, Blocks 4 open bugs)

Details

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
Also 
resource-selection-pointer-insert-source.html, resource-selection-pointer-remove-source.html resource-selection-source-media.html

Those are related to </video> tag.
(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.
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)
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)
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>.
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
This is basically a dup of bug 1318316, which depends on the promise+microtask scheduling stuff I've been doing.
According to Smaug's comment, that should be done by smaug.
Flags: needinfo?(ctai)
Should we just dupe this to bug 1318316, then?
Flags: needinfo?(bugs)
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)
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>?
There are no macro tasks ;)

But HTML spec defines that microtask checkpoint happens before <script> execution.
This doesn't sound something we plan to ship in this release, re-set the priority.
Priority: P1 → P3
Component: DOM → DOM: Core & HTML
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.