Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

editorial: Define a task source for the tasks run by this specification. #301

Merged
merged 1 commit into from
Feb 9, 2021

Conversation

rakuco
Copy link
Member

@rakuco rakuco commented Feb 8, 2021

This allows implementations to better decide when to run events and tasks
from this spec in relation to others, and specifying a task source is a
requirement when queuing tasks like we do [1].

[1] https://html.spec.whatwg.org/multipage/webappapis.html#event-loop-for-spec-authors

Rather than picking a generic task source from the HTML spec, we define a
new task source, the "screen wake lock task source", following the current
guidance in w3ctag/design-principles#38 (which is still open).

Implementation commitment:


Preview | Diff

@rakuco rakuco requested review from reillyeon and kenchris February 8, 2021 10:12
This allows implementations to better decide when to run events and tasks
from this spec in relation to others, and specifying a task source is a
requirement when queuing tasks like we do [1].

[1] https://html.spec.whatwg.org/multipage/webappapis.html#event-loop-for-spec-authors

Rather than picking a generic task source from the HTML spec, we define a
new task source, the "screen wake lock task source", following the current
guidance in w3ctag/design-principles#38 (which is still open).
@rakuco rakuco force-pushed the editorial/define-task-source branch from 83d8dc8 to fa8693f Compare February 8, 2021 15:57
@rakuco rakuco merged commit 9b8a271 into w3c:gh-pages Feb 9, 2021
@rakuco rakuco deleted the editorial/define-task-source branch February 9, 2021 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants