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

Unified Diff: base/trace_event/event_filter_registry_unittest.cc

Issue 2499583003: [OBSOLETE] tracing: split trace event filter logic out of TraceLog (Closed)
Patch Set: . Created 4 years, 1 month 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: base/trace_event/event_filter_registry_unittest.cc
diff --git a/base/trace_event/event_filter_registry_unittest.cc b/base/trace_event/event_filter_registry_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bc9a8fdf4d1004a4bc1a9922ba15462590c0d200
--- /dev/null
+++ b/base/trace_event/event_filter_registry_unittest.cc
@@ -0,0 +1,63 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/trace_event/event_filter_registry.h"
+
+#include <string>
+
+#include "base/trace_event/trace_event_filter.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace base {
+namespace trace_event {
+
+class TestFilter1 : public TraceEventFilter {
+ public:
+ static const char kName[];
+
+ TestFilter1() {}
+ ~TestFilter1() override {}
+
+ bool FilterTraceEvent(const TraceEvent&) const override { return true; }
+ const char* name() const override { return kName; }
+};
+const char TestFilter1::kName[] = "test_filter1";
+
+class TestFilter2 : public TraceEventFilter {
+ public:
+ static const char kName[];
+
+ TestFilter2() {}
+ ~TestFilter2() override {}
+
+ bool FilterTraceEvent(const TraceEvent&) const override { return true; }
+ const char* name() const override { return kName; }
+};
+const char TestFilter2::kName[] = "test_filter2";
+
+TEST(TraceEventFilterRegistryTest, RegistryTest) {
+ auto* filter1 = EventFilterRegistry::RegisterFilterLocked<TestFilter1>();
+ EXPECT_NE(nullptr, filter1);
+
+ auto* filter2 = EventFilterRegistry::RegisterFilterLocked<TestFilter2>();
+ EXPECT_NE(nullptr, filter2);
+ EXPECT_NE(static_cast<TraceEventFilter*>(filter1),
+ static_cast<TraceEventFilter*>(filter2));
+
+ std::string copied_name1(TestFilter1::kName);
+ EXPECT_EQ(filter1, EventFilterRegistry::GetByName(copied_name1.c_str()));
+ const int index1 = EventFilterRegistry::GetIndexByName(TestFilter1::kName);
+ EXPECT_GE(index1, 0);
+ EXPECT_EQ(filter1, EventFilterRegistry::Get(index1));
+
+ std::string copied_name2(TestFilter2::kName);
+ EXPECT_EQ(filter2, EventFilterRegistry::GetByName(copied_name2.c_str()));
+ const int index2 = EventFilterRegistry::GetIndexByName(copied_name2.c_str());
+ EXPECT_GE(index2, 0);
+ EXPECT_NE(index1, index2);
+ EXPECT_EQ(filter2, EventFilterRegistry::Get(index2));
+}
+
+} // namespace trace_event
+} // namespace base

Powered by Google App Engine
This is Rietveld 408576698