StableState should be fired before second <script> tag and right after the </video>.

NEW
Unassigned

Status

()

Core
DOM
P3
normal
a year ago
10 months ago

People

(Reporter: ctai, Unassigned)

Tracking

(Depends on: 1 bug, Blocks: 11 bugs)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 affected)

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
Blocks: 1117306
Duplicate of this bug: 1117306
Blocks: 1152018
Duplicate of this bug: 1152018
Blocks: 1218518
Duplicate of this bug: 1218518
Blocks: 1224885
Duplicate of this bug: 1224885
Blocks: 1221399
Duplicate of this bug: 1221399
Blocks: 1298166
Duplicate of this bug: 1298166
Blocks: 1247844
Duplicate of this bug: 1247844
Blocks: 1189800
Duplicate of this bug: 1189800
Blocks: 1107766
Duplicate of this bug: 1107766
Blocks: 1122741
Duplicate of this bug: 1122741
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)

Updated

a year 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>.
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.
Duplicate of this bug: 1117306
Duplicate of this bug: 1152018
Duplicate of this bug: 1218518
Duplicate of this bug: 1224885
Duplicate of this bug: 1221399
Duplicate of this bug: 1298166
Duplicate of this bug: 1247844
Duplicate of this bug: 1189800
Duplicate of this bug: 1107766
Duplicate of this bug: 1122741
Duplicate of this bug: 1147952
Duplicate of this bug: 1318221

Comment 35

10 months ago
This doesn't sound something we plan to ship in this release, re-set the priority.
Priority: P1 → P3
You need to log in before you can comment on or make changes to this bug.