Moved Opus-specific payload splitting into AudioDecoderOpus.

webrtc/modules/audio_coding/neteq/packet_buffer.cc

 /*
  *  Copyright (c) 2012 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 244 matching lines @@
   }
   return 0;
 }
 
 int PacketBuffer::DiscardAllOldPackets(uint32_t timestamp_limit) {
   return DiscardOldPackets(timestamp_limit, 0);
 }
 
 void PacketBuffer::DiscardPacketsWithPayloadType(uint8_t payload_type) {
   for (auto it = buffer_.begin(); it != buffer_.end(); /* */) {
     Packet *packet = *it;

[hlundin-webrtc, 2016/09/20 07:02:24]

Packet* packet

[ossu, 2016/09/20 13:51:56]

Acknowledged.

     if (packet->header.payloadType == payload_type) {
       delete packet;
       it = buffer_.erase(it);
     } else {
       ++it;
     }
   }

[kwiberg-webrtc, 2016/09/20 09:14:45]

Can you use std::remove_if instead? It should be shorter and simpler, and run in
time O(n) instead of O(n^2).

[ossu, 2016/09/20 13:51:56]

Since buffer_ is a std::list, this runs in O(n) and having to do delete manually
precludes using remove_if. I might revisit this if/when I take a look at Packet
handling in NetEq again. It'd be nice if we didn't have to manually
memory-manage them.

[kwiberg-webrtc, 2016/09/20 14:56:23]

Aha, an std::list. And yes, the manual memory management. OK.

 }
 
 size_t PacketBuffer::NumPacketsInBuffer() const {
   return buffer_.size();
 }
 
 size_t PacketBuffer::NumSamplesInBuffer(size_t last_decoded_length) const {
   size_t num_samples = 0;
   size_t last_duration = last_decoded_length;
   for (Packet* packet : buffer_) {
     if (packet->frame) {
-      if (!packet->primary) {
+      // TODO(hlundin): Verify that it's fine to count all packets and remove
+      // this check.
+      if (packet->priority != Packet::Priority(0, 0)) {
         continue;
       }
       size_t duration = packet->frame->Duration();
       if (duration > 0) {
         last_duration = duration;  // Save the most up-to-date (valid) duration.
       }
     }
     num_samples += last_duration;
   }
   return num_samples;
@@ skipping 14 matching lines @@
     // Continue while the list is not empty.
   }
 }
 
 void PacketBuffer::BufferStat(int* num_packets, int* max_num_packets) const {
   *num_packets = static_cast<int>(buffer_.size());
   *max_num_packets = static_cast<int>(max_number_of_packets_);
 }
 
 }  // namespace webrtc