webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h - Issue 1765443002: Added log messages for important call setup events: first packet sent/received

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

Issue 1765443002: Added log messages for important call setup events: first packet sent/received (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master

Patch Set: Replaced ad-hoc boolean flags with the new class OneTimeEvent Created 4 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

« webrtc/base/onetimeevent.h ('K') | « webrtc/base/onetimeevent.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.cc » ('j') | webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

diff --git a/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h b/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

index 1dd07d1cc923127f141046183b0ee5069e62c014..ab3bbad25867af93660e6beac3609562e3beb0f1 100644

--- a/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

+++ b/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h

@@ -13,6 +13,7 @@

#include <set>

+#include "webrtc/base/onetimeevent.h"

#include "webrtc/base/scoped_ptr.h"

#include "webrtc/modules/rtp_rtcp/include/rtp_receiver.h"

#include "webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h"

@@ -121,6 +122,7 @@ class RTPReceiverAudio : public RTPReceiverStrategy,

uint8_t current_remote_energy_[kRtpCsrcSize];

RtpAudioFeedback* cb_audio_feedback_;

+ OneTimeEvent first_packet_received_;

pthatcher1 2016/03/25 22:35:23 This one could be thread-unsafe, right?

skvlad 2016/03/26 02:00:08 Looks like it cannot be - from the looks of it, th

};

} // namespace webrtc

« webrtc/base/onetimeevent.h ('K') | « webrtc/base/onetimeevent.h ('k') | webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.cc » ('j') | webrtc/modules/rtp_rtcp/source/rtp_sender_audio.h » ('J')