Add RTP RID test to the simulcast mochitest

RESOLVED FIXED in Firefox 58

Status

()

P3
normal
Rank:
25
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: ng, Assigned: ng)

Tracking

unspecified
mozilla58
Points:
---

Firefox Tracking Flags

(firefox58 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
Add a test to ensure that RTP Stream ID (RID) is being emitted correctly in the simulcast mochitest using the new packet dump superpowers.
Comment hidden (mozreview-request)
Rank: 25
Priority: -- → P2
(Assignee)

Updated

a year ago
Attachment #8882523 - Flags: review?(drno)

Comment 2

a year ago
mozreview-review
Comment on attachment 8882523 [details]
Bug 1377420 - add RID check in RTP in simulcast mochitests

https://reviewboard.mozilla.org/r/153644/#review166386

::: dom/media/tests/mochitest/parser_rtp.js:51
(Diff revision 1)
> +  // Marker             1 Bit
> +  header.marker =  (0x80 & byte) >> 7;
> +  // Payload Type       7 Bit
> +  header.payloadType = (0x7F & byte);
> +  // Sequence Number   16 Bit
> +  header.sequenceNumber = view.getUint16(offset);

As you haven't increased the offset before this I think you are re-reading the marker and payload byte as the sequence number here.
Attachment #8882523 - Flags: review?(drno) → review+
(Assignee)

Comment 3

a year ago
mozreview-review-reply
Comment on attachment 8882523 [details]
Bug 1377420 - add RID check in RTP in simulcast mochitests

https://reviewboard.mozilla.org/r/153644/#review166386

> As you haven't increased the offset before this I think you are re-reading the marker and payload byte as the sequence number here.

I believe I am incrementing it at line 45 (and 34 previously).
Comment hidden (mozreview-request)
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Comment hidden (mozreview-request)

Comment 7

a year ago
Pushed by na-g@nostrum.com:
https://hg.mozilla.org/integration/autoland/rev/b6fc02a83622
add RID check in RTP in simulcast mochitests r=drno

Comment 8

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b6fc02a83622
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox58: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.