| OLD | NEW |
| 1 /* | 1 /* |
| 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. | 2 * Copyright (c) 2015 The WebRTC project authors. All Rights Reserved. |
| 3 * | 3 * |
| 4 * Use of this source code is governed by a BSD-style license | 4 * Use of this source code is governed by a BSD-style license |
| 5 * that can be found in the LICENSE file in the root of the source | 5 * that can be found in the LICENSE file in the root of the source |
| 6 * tree. An additional intellectual property rights grant can be found | 6 * tree. An additional intellectual property rights grant can be found |
| 7 * in the file PATENTS. All contributing project authors may | 7 * in the file PATENTS. All contributing project authors may |
| 8 * be found in the AUTHORS file in the root of the source tree. | 8 * be found in the AUTHORS file in the root of the source tree. |
| 9 */ | 9 */ |
| 10 | 10 |
| (...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 82 RTC_DCHECK(packet_router_ != nullptr); | 82 RTC_DCHECK(packet_router_ != nullptr); |
| 83 packet_router_->SendFeedback(&feedback_packet); | 83 packet_router_->SendFeedback(&feedback_packet); |
| 84 } else { | 84 } else { |
| 85 more_to_build = false; | 85 more_to_build = false; |
| 86 } | 86 } |
| 87 } | 87 } |
| 88 } | 88 } |
| 89 | 89 |
| 90 void RemoteEstimatorProxy::OnPacketArrival(uint16_t sequence_number, | 90 void RemoteEstimatorProxy::OnPacketArrival(uint16_t sequence_number, |
| 91 int64_t arrival_time) { | 91 int64_t arrival_time) { |
| 92 // TODO(holmer): We should handle a backwards wrap here if the first |
| 93 // sequence number was small and the new sequence number is large. The |
| 94 // SequenceNumberUnwrapper doesn't do this, so we should replace this with |
| 95 // calls to IsNewerSequenceNumber instead. |
| 92 int64_t seq = unwrapper_.Unwrap(sequence_number); | 96 int64_t seq = unwrapper_.Unwrap(sequence_number); |
| 97 if (seq > window_start_seq_ + 0xFFFF / 2) { |
| 98 LOG(LS_WARNING) << "Skipping this sequence number (" << sequence_number |
| 99 << ") since it likely is reordered, but the unwrapper" |
| 100 "failed to handle it. Feedback window starts at " |
| 101 << window_start_seq_ << "."; |
| 102 return; |
| 103 } |
| 93 | 104 |
| 94 if (packet_arrival_times_.lower_bound(window_start_seq_) == | 105 if (packet_arrival_times_.lower_bound(window_start_seq_) == |
| 95 packet_arrival_times_.end()) { | 106 packet_arrival_times_.end()) { |
| 96 // Start new feedback packet, cull old packets. | 107 // Start new feedback packet, cull old packets. |
| 97 for (auto it = packet_arrival_times_.begin(); | 108 for (auto it = packet_arrival_times_.begin(); |
| 98 it != packet_arrival_times_.end() && it->first < seq && | 109 it != packet_arrival_times_.end() && it->first < seq && |
| 99 arrival_time - it->second >= kBackWindowMs;) { | 110 arrival_time - it->second >= kBackWindowMs;) { |
| 100 auto delete_it = it; | 111 auto delete_it = it; |
| 101 ++it; | 112 ++it; |
| 102 packet_arrival_times_.erase(delete_it); | 113 packet_arrival_times_.erase(delete_it); |
| (...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 153 // Note: Don't erase items from packet_arrival_times_ after sending, in case | 164 // Note: Don't erase items from packet_arrival_times_ after sending, in case |
| 154 // they need to be re-sent after a reordering. Removal will be handled | 165 // they need to be re-sent after a reordering. Removal will be handled |
| 155 // by OnPacketArrival once packets are too old. | 166 // by OnPacketArrival once packets are too old. |
| 156 window_start_seq_ = it->first + 1; | 167 window_start_seq_ = it->first + 1; |
| 157 } | 168 } |
| 158 | 169 |
| 159 return true; | 170 return true; |
| 160 } | 171 } |
| 161 | 172 |
| 162 } // namespace webrtc | 173 } // namespace webrtc |
| OLD | NEW |