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

Unified Diff: webrtc/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc

Issue 2995363002: Replace gflags usages with rtc_base/flags in all targets based on test_main (Closed)
Patch Set: Fix string use after free Created 3 years, 4 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/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc
diff --git a/webrtc/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc b/webrtc/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc
index 9af0eeca0b24a20beb258ad329d408b7b79d76b3..136c13bdfe7c65d0cfa22984d5c79c7c19b9f5bc 100644
--- a/webrtc/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc
+++ b/webrtc/voice_engine/test/auto_test/standard/rtp_rtcp_test.cc
@@ -10,6 +10,7 @@
#include <memory>
+#include "gflags/gflags.h"
#include "webrtc/rtc_base/criticalsection.h"
#include "webrtc/system_wrappers/include/atomic32.h"
#include "webrtc/system_wrappers/include/event_wrapper.h"
@@ -17,6 +18,8 @@
#include "webrtc/voice_engine/test/auto_test/fixtures/after_streaming_fixture.h"
#include "webrtc/voice_engine/test/auto_test/voe_standard_test.h"
+DECLARE_bool(include_timing_dependent_tests);
kjellander_webrtc 2017/08/24 07:31:34 I see this is moved from webrtc/voice_engine/test/
oprypin_webrtc 2017/08/24 08:29:56 This is not a target based on test_main, I just wa
+
class TestRtpObserver : public webrtc::VoERTPObserver {
public:
TestRtpObserver() : changed_ssrc_event_(webrtc::EventWrapper::Create()) {}

Powered by Google App Engine
This is Rietveld 408576698