-
Notifications
You must be signed in to change notification settings - Fork 3
Restore persistent-usage-record and update editors #1
Conversation
Update editors to add w3cid
encrypted-media-respec.html
Outdated
@@ -1463,6 +1468,48 @@ <h2><dfn>MediaKeys</dfn> Interface</h2> | |||
See <a def-id="session-storage"></a>. | |||
</p> | |||
</td></tr> | |||
<tr><td><dfn><code id="idl-def-MediaKeySessionType.persistent-usage-record">persistent-usage-record</code></dfn></td><td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit, should be:
<code><dfn>persistent-usage-record</dfn></code>
Wrapping in code
allows ReSpec to automatically put code around any:
<a>persistent-usage-record</a>
Also, it's probably not a good idea to try to recreate ReSpec's generated ids, as they can change.
Same applies for the rest.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, thanks.
I guess the ReSpec ids are there because at one point something wasn't working without them. I'd prefer to remove them as a separate step, since they are there throughout the spec right now. It should probably be done in the upstream repo also.
Does anyone have further comments on this PR ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Preview | Diff