Chromium Code Reviews| Index: webrtc/modules/audio_coding/neteq/packet_buffer_unittest.cc | 
| diff --git a/webrtc/modules/audio_coding/neteq/packet_buffer_unittest.cc b/webrtc/modules/audio_coding/neteq/packet_buffer_unittest.cc | 
| index 1b86d8326b6af182b925540fcdec7d22ab49682f..52cd6bbdb4ad5e32fc95cadf0fdf6ebd1ebb41a2 100644 | 
| --- a/webrtc/modules/audio_coding/neteq/packet_buffer_unittest.cc | 
| +++ b/webrtc/modules/audio_coding/neteq/packet_buffer_unittest.cc | 
| @@ -60,7 +60,7 @@ Packet* PacketGenerator::NextPacket(int payload_size_bytes) { | 
| packet->header.ssrc = 0x12345678; | 
| packet->header.numCSRCs = 0; | 
| packet->header.paddingLength = 0; | 
| - packet->primary = true; | 
| + packet->priority = Packet::kHighestPriority; | 
| 
 
kwiberg-webrtc
2016/09/20 09:14:45
Why can you skip this?
 
ossu
2016/09/20 13:51:56
Because Priority is now default constructed at the
 
kwiberg-webrtc
2016/09/20 14:56:22
Acknowledged.
 
 | 
| packet->payload.SetSize(payload_size_bytes); | 
| ++seq_no_; | 
| ts_ += frame_size_; | 
| @@ -284,7 +284,7 @@ TEST(PacketBuffer, ExtractOrderRedundancy) { | 
| packet_facts[i].payload_type, | 
| kFrameSize); | 
| Packet* packet = gen.NextPacket(kPayloadLength); | 
| - packet->primary = packet_facts[i].primary; | 
| + packet->priority.red_level = packet_facts[i].primary ? 0 : 1; | 
| EXPECT_EQ(PacketBuffer::kOK, buffer.InsertPacket(packet)); | 
| if (packet_facts[i].extract_order >= 0) { | 
| expect_order[packet_facts[i].extract_order] = packet; | 
| @@ -558,8 +558,8 @@ TEST(PacketBuffer, ComparePackets) { | 
| // Test equal timestamps and sequence numbers, but only 'b' is primary. | 
| a->header.sequenceNumber = b->header.sequenceNumber; | 
| - a->primary = false; | 
| - b->primary = true; | 
| + a->priority = {1, 0}; | 
| 
 
hlundin-webrtc
2016/09/15 09:33:30
Add tests for the other cases too.
 
ossu
2016/09/15 12:22:53
Acknowledged.
 
 | 
| + b->priority = {0, 0}; | 
| EXPECT_FALSE(*a == *b); | 
| EXPECT_TRUE(*a != *b); | 
| EXPECT_FALSE(*a < *b); |