Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(205)

Unified Diff: webrtc/modules/congestion_controller/transport_feedback_adapter.h

Issue 2638083002: Attach TransportFeedbackPacketLossTracker to ANA (PLR only) (Closed)
Patch Set: Fix UT Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/congestion_controller/transport_feedback_adapter.h
diff --git a/webrtc/modules/congestion_controller/transport_feedback_adapter.h b/webrtc/modules/congestion_controller/transport_feedback_adapter.h
index 616bebe00a03fd76b29f559835f1399bc8cbbbc9..2e70e448da9c7c6449c8d74ac8a9b139aafda11a 100644
--- a/webrtc/modules/congestion_controller/transport_feedback_adapter.h
+++ b/webrtc/modules/congestion_controller/transport_feedback_adapter.h
@@ -21,6 +21,8 @@
namespace webrtc {
+class PacketFeedbackObserver;
+
namespace rtcp {
class TransportFeedback;
} // namespace rtcp
@@ -30,7 +32,11 @@ class TransportFeedbackAdapter {
explicit TransportFeedbackAdapter(const Clock* clock);
virtual ~TransportFeedbackAdapter();
- void AddPacket(uint16_t sequence_number,
+ void RegisterPacketFeedbackObserver(PacketFeedbackObserver* observer);
+ void DeRegisterPacketFeedbackObserver(PacketFeedbackObserver* observer);
+
+ void AddPacket(uint32_t ssrc,
+ uint16_t sequence_number,
size_t length,
const PacedPacketInfo& pacing_info);
void OnSentPacket(uint16_t sequence_number, int64_t send_time_ms);
@@ -57,8 +63,11 @@ class TransportFeedbackAdapter {
int64_t current_offset_ms_;
int64_t last_timestamp_us_;
std::vector<PacketFeedback> last_packet_feedback_vector_;
- uint16_t local_net_id_;
- uint16_t remote_net_id_;
+ uint16_t local_net_id_ GUARDED_BY(&lock_);
+ uint16_t remote_net_id_ GUARDED_BY(&lock_);
+
+ rtc::CriticalSection observers_lock_;
+ std::vector<PacketFeedbackObserver*> observers_ GUARDED_BY(&observers_lock_);
};
} // namespace webrtc

Powered by Google App Engine
This is Rietveld 408576698