Skip to content

Commit

Permalink
Merge pull request #208 from w3c/fix-serialization
Browse files Browse the repository at this point in the history
Remove (de)serialization of no longer existent RTCEncodedVideoFrame.timestamp
  • Loading branch information
alvestrand committed Oct 9, 2023
2 parents e258de1 + 3041a21 commit 7721c15
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions index.bs
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ dictionary RTCEncodedVideoFrameMetadata {
</p>
</dd>
<dt>
<dfn>timestamp</dfn> <span class=
<dfn dict-member>timestamp</dfn> <span class=
"idlMemberType">long long</span>
</dt>
<dd>
Expand Down Expand Up @@ -449,14 +449,12 @@ Their [=serialization steps=], given |value|, |serialized|, and |forStorage|, ar

1. If |forStorage| is true, then throw a {{DataCloneError}}.
1. Set |serialized|.`[[type]]` to the value of |value|.{{RTCEncodedVideoFrame/type}}
1. Set |serialized|.`[[timestamp]]` to the value of |value|.{{RTCEncodedVideoFrame/timestamp}}
1. Set |serialized|.`[[metadata]]` to an internal representation of |value|'s metadata.
1. Set |serialized|.`[[data]]` to |value|.`[[data]]`

Their [=deserialization steps=], given |serialized|, |value| and |realm|, are:

1. Set |value|.{{RTCEncodedVideoFrame/type}} to |serialized|.`[[type]]`
1. Set |value|.{{RTCEncodedVideoFrame/timestamp}} to |serialized|.`[[timestamp]]`
1. Set |value|'s metadata to the platform object representation of |serialized|.`[[metadata]]`
1. Set |value|.`[[data]]` to |serialized|.`[[data]]`.

Expand Down Expand Up @@ -569,13 +567,11 @@ interface RTCEncodedAudioFrame {
Their [=serialization steps=], given |value|, |serialized|, and |forStorage|, are:

1. If |forStorage| is true, then throw a {{DataCloneError}}.
1. Set |serialized|.`[[timestamp]]` to the value of |value|.{{RTCEncodedAudioFrame/timestamp}}
1. Set |serialized|.`[[metadata]]` to an internal representation of |value|'s metadata.
1. Set |serialized|.`[[data]]` to |value|.`[[data]]`

Their [=deserialization steps=], given |serialized|, |value| and |realm|, are:

1. Set |value|.{{RTCEncodedAudioFrame/timestamp}} to |serialized|.`[[timestamp]]`
1. Set |value|'s metadata to the platform object representation of |serialized|.`[[metadata]]`
1. Set |value|.`[[data]]` to |serialized|.`[[data]]`.

Expand Down

0 comments on commit 7721c15

Please sign in to comment.