Add directive to make TRACE_EVENT macros optional.
Bug: webrtc:11132
Change-Id: I801994ad262e1acff73e4c20afd7a7343b56268c
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/160654
Commit-Queue: Doudou Kisabaka <doudouk@google.com>
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#29949}
diff --git a/rtc_base/event_tracer_unittest.cc b/rtc_base/event_tracer_unittest.cc
index 2556d24..79cc9c0 100644
--- a/rtc_base/event_tracer_unittest.cc
+++ b/rtc_base/event_tracer_unittest.cc
@@ -45,22 +45,6 @@
int events_logged_ RTC_GUARDED_BY(crit_) = 0;
};
-const unsigned char* GetCategoryEnabledHandler(const char* /*name*/) {
- return reinterpret_cast<const unsigned char*>("test");
-}
-
-void TraceEventHandler(char /*phase*/,
- const unsigned char* /*category_enabled*/,
- const char* /*name*/,
- unsigned long long /*id*/,
- int /*num_args*/,
- const char** /*arg_names*/,
- const unsigned char* /*arg_types*/,
- const unsigned long long* /*arg_values*/,
- unsigned char /*flags*/) {
- TestStatistics::Get()->Increment();
-}
-
} // namespace
namespace webrtc {
@@ -71,11 +55,27 @@
TestStatistics::Get()->Reset();
}
+#if RTC_TRACE_EVENTS_ENABLED
TEST(EventTracerTest, ScopedTraceEvent) {
- SetupEventTracer(&GetCategoryEnabledHandler, &TraceEventHandler);
+ SetupEventTracer(
+ [](const char* /*name*/) {
+ return reinterpret_cast<const unsigned char*>("test");
+ },
+ [](char /*phase*/,
+ const unsigned char* /*category_enabled*/,
+ const char* /*name*/,
+ unsigned long long /*id*/,
+ int /*num_args*/,
+ const char** /*arg_names*/,
+ const unsigned char* /*arg_types*/,
+ const unsigned long long* /*arg_values*/,
+ unsigned char /*flags*/) {
+ TestStatistics::Get()->Increment();
+ });
{ TRACE_EVENT0("test", "ScopedTraceEvent"); }
EXPECT_EQ(2, TestStatistics::Get()->Count());
TestStatistics::Get()->Reset();
}
+#endif
} // namespace webrtc