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

Unified Diff: webrtc/audio/audio_send_stream.h

Issue 2638083002: Attach TransportFeedbackPacketLossTracker to ANA (PLR only) (Closed)
Patch Set: Rebased 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/audio/audio_send_stream.h
diff --git a/webrtc/audio/audio_send_stream.h b/webrtc/audio/audio_send_stream.h
index 5ee49da91a7a1469285d1fcf675c39c53a43ac5c..0a4fc9da4d534ffb6d5fe22fe0a796ff834280f8 100644
--- a/webrtc/audio/audio_send_stream.h
+++ b/webrtc/audio/audio_send_stream.h
@@ -18,6 +18,8 @@
#include "webrtc/call/audio_send_stream.h"
#include "webrtc/call/audio_state.h"
#include "webrtc/call/bitrate_allocator.h"
+#include "webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h"
+#include "webrtc/voice_engine/transport_feedback_packet_loss_tracker.h"
namespace webrtc {
class CongestionController;
@@ -33,7 +35,8 @@ class ChannelProxy;
namespace internal {
class AudioSendStream final : public webrtc::AudioSendStream,
- public webrtc::BitrateAllocatorObserver {
+ public webrtc::BitrateAllocatorObserver,
+ public webrtc::TransportFeedbackAdapterObserver {
public:
AudioSendStream(const webrtc::AudioSendStream::Config& config,
const rtc::scoped_refptr<webrtc::AudioState>& audio_state,
@@ -62,6 +65,11 @@ class AudioSendStream final : public webrtc::AudioSendStream,
int64_t rtt,
int64_t probing_interval_ms) override;
+ // From TransportFeedbackAdapterObserver
+ void OnPacketAdded(uint32_t ssrc, uint16_t seq_num) override;
+ void OnNewTransportFeedbacks(
+ const std::vector<PacketFeedback>& packet_feedbacks) override;
+
const webrtc::AudioSendStream::Config& config() const;
void SetTransportOverhead(int transport_overhead_per_packet);
@@ -70,6 +78,7 @@ class AudioSendStream final : public webrtc::AudioSendStream,
bool SetupSendCodec();
+ const Clock* const clock_;
rtc::ThreadChecker thread_checker_;
rtc::TaskQueue* worker_queue_;
const webrtc::AudioSendStream::Config config_;
@@ -80,6 +89,10 @@ class AudioSendStream final : public webrtc::AudioSendStream,
CongestionController* const congestion_controller_;
std::unique_ptr<RtcpBandwidthObserver> bandwidth_observer_;
+ rtc::CriticalSection packet_loss_tracker_cs_;
+ TransportFeedbackPacketLossTracker packet_loss_tracker_
+ GUARDED_BY(&packet_loss_tracker_cs_);
+
RTC_DISALLOW_IMPLICIT_CONSTRUCTORS(AudioSendStream);
};
} // namespace internal
« no previous file with comments | « no previous file | webrtc/audio/audio_send_stream.cc » ('j') | webrtc/modules/congestion_controller/transport_feedback_adapter.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698