Conform 'current time' definition to the current hr-time spec #236
Conversation
| the current moment in time.</p> | ||
| <p>The term <dfn>current time</dfn> refers to the result of running | ||
| <a data-cite="HR-TIME-ED#dfn-current-high-resolution-time">current high | ||
| resolution time</a> where |current global| is the {{Window}} object.</p> |
There was a problem hiding this comment.
Maybe you can add an inline issue (<p class=issue> IIRC) to align this to the published spec, once we've actually published it
There was a problem hiding this comment.
Probably not needed as this will be on L3, right ?
| <p>The term <dfn>current time</dfn> refers to the number of | ||
| milliseconds since the start of navigation of the document until | ||
| the current moment in time.</p> | ||
| <p>The term <dfn>current time</dfn> refers to the result of running |
There was a problem hiding this comment.
I think we'd be better off eliminating this dfn, and call the algorithm directly with the current global object from the few places in the processing model that are calling it.
The reason for that is that we need to call it with a specific global object, and this dfn doesn't give us that. Does that make sense?
There was a problem hiding this comment.
Sounds good and also seems more robust since I have observed that the term current time is mostly used in algorithmic steps in this spec.
So would replacing occurences like:
record the <a>current time</a> in <a>startTime</a>
with
record the result of <a data-cite="HR-TIME-ED#dfn-current-high-resolution-time">current high resolution time</a> where |current global| is {{Window}} in <a>startTime</a>
fit the case ?
There was a problem hiding this comment.
{{Window}} is a reference to the Window object, but not to a specific window.
You probably want to reference the current global object
|
@igneel64 - Friendly ping! I'd love to help you land this.. |
Conform 'current time' definition to the current hr-time spec current high resolution time.
Added
HR-TIME-EDlocalBiblio entry.Preview | Diff