-
Notifications
You must be signed in to change notification settings - Fork 46
Editorial: Clean up spec markdown #75
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,4 @@ | ||
Introduction {#intro} | ||
===================== | ||
# Introduction # {#intro} | ||
|
||
<div class="non-normative"> | ||
|
||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,58 +1,50 @@ | ||
Modifications to Other Standards {#sec-patches} | ||
===================== | ||
# Modifications to Other Standards # {#sec-patches} | ||
|
||
The HTML Standard {#sec-patches-html} | ||
--------------------- | ||
## The HTML Standard ## {#sec-patches-html} | ||
|
||
### `WindowOrWorkerGlobalScope` ### {#sec-patches-html-windoworworkerglobalscope} | ||
|
||
Each object implementing the {{WindowOrWorkerGlobalScope}} mixin has a | ||
corresponding <dfn for="WindowOrWorkerGlobalScope">scheduler</dfn>, which | ||
is initialized as a new {{Scheduler}}. | ||
Each object implementing the {{WindowOrWorkerGlobalScope}} mixin has a corresponding | ||
<dfn for="WindowOrWorkerGlobalScope">scheduler</dfn>, which is initialized as a new {{Scheduler}}. | ||
|
||
<pre class='idl'> | ||
partial interface mixin WindowOrWorkerGlobalScope { | ||
[Replaceable] readonly attribute Scheduler scheduler; | ||
}; | ||
</pre> | ||
|
||
The <dfn attribute for="WindowOrWorkerGlobalScope">scheduler</dfn> attribute's | ||
getter steps are to return [=this=]'s [=WindowOrWorkerGlobalScope/scheduler=]. | ||
|
||
The <dfn attribute for="WindowOrWorkerGlobalScope">scheduler</dfn> attribute's getter steps are to | ||
return [=this=]'s [=WindowOrWorkerGlobalScope/scheduler=]. | ||
|
||
### <a href="https://html.spec.whatwg.org/multipage/webappapis.html#definitions-3">Event loop: definitions</a> ### {#sec-patches-html-event-loop-definitions} | ||
|
||
Replace: For each [=event loop=], every [=task source=] must be associated with | ||
a specific [=task queue=]. | ||
Replace: For each [=event loop=], every [=task source=] must be associated with a specific [=task | ||
queue=]. | ||
|
||
With: For each [=event loop=], every [=task source=] that is not a | ||
[=scheduler task source=] must be associated with a specific [=task queue=]. | ||
With: For each [=event loop=], every [=task source=] that is not a [=scheduler task source=] must be | ||
associated with a specific [=task queue=]. | ||
|
||
### <a href="https://html.spec.whatwg.org/multipage/webappapis.html#event-loop-processing-model">Event loop: processing model</a> ### {#sec-patches-html-event-loop-processing} | ||
|
||
Add the following steps to the event loop processing steps, before step 1: | ||
|
||
1. Let |queues| be the [=set=] of the [=event loop=]'s [=task queues=] that | ||
contain at least one <a for="task">runnable</a> <a for="/">task</a>. | ||
1. Let |schedulers| be the [=set=] of all {{Scheduler}} objects whose | ||
[=relevant agent's=] [=agent/event loop=] is this event loop and that | ||
[=have a runnable task=]. | ||
1. If |schedulers| and |queues| are both [=list/empty=], skip to the | ||
<code>microtasks</code> step below. | ||
1. Let |queues| be the [=set=] of the [=event loop=]'s [=task queues=] that contain at least one | ||
<a for="task">runnable</a> <a for="/">task</a>. | ||
shaseley marked this conversation as resolved.
Show resolved
Hide resolved
|
||
1. Let |schedulers| be the [=set=] of all {{Scheduler}} objects whose [=relevant agent's=] | ||
[=agent/event loop=] is this event loop and that [=have a runnable task=]. | ||
1. If |schedulers| and |queues| are both [=list/empty=], skip to the <code>microtasks</code> step | ||
below. | ||
|
||
Modify step 1 to read: | ||
|
||
1. Let |taskQueue| be one of the following, chosen in an | ||
[=implementation-defined=] manner: | ||
* If |queues| is not [=list/empty=], one of the [=task queues=] in |queues|, | ||
chosen in an [=implementation-defined=] manner. | ||
* If |schedulers| is not [=list/empty=], the result of | ||
[=selecting the task queue of the next scheduler task=] from one of the | ||
{{Scheduler}}s in |schedulers|, chosen in an [=implementation-defined=] | ||
manner. | ||
|
||
Issue: The |taskQueue| in this step will either be a [=set=] of [=tasks=] or a | ||
[=set=] of [=scheduler tasks=]. The steps that follow only [=set/remove=] an | ||
[=set/item=], so they are *roughly* compatible. Ideally, there would be a | ||
common task queue interface that supports a `pop()` method that would return a | ||
plain [=task=], but that would invlove a fair amount of refactoring. | ||
1. Let |taskQueue| be one of the following, chosen in an [=implementation-defined=] manner: | ||
* If |queues| is not [=list/empty=], one of the [=task queues=] in |queues|, chosen in an | ||
[=implementation-defined=] manner. | ||
* If |schedulers| is not [=list/empty=], the result of [=selecting the task queue of the next | ||
scheduler task=] from one of the {{Scheduler}}s in |schedulers|, chosen in an | ||
[=implementation-defined=] manner. | ||
|
||
Issue: The |taskQueue| in this step will either be a [=set=] of [=tasks=] or a [=set=] of | ||
[=scheduler tasks=]. The steps that follow only [=set/remove=] an [=set/item=], so they are | ||
*roughly* compatible. Ideally, there would be a common task queue interface that supports a `pop()` | ||
method that would return a plain [=task=], but that would invlove a fair amount of refactoring. | ||
shaseley marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.