Trace Redaction - Use strategy pattern for ftrace events
Following changes else where in the code base, this changes the
top-level ftrace event primitive to use the strategy pattern
to filter and write ftrace events to the new packet.
The "write" function will be used to change the ftrace event pid
during thread merging.
The old remapping primitives had to be removed from the build files
because this change conflicted with them (class designs). Because
these files/classes were going to be revisited, this cost was
accessible.
The ReturnErrorForEmptyAllowList test was removed because it is no
longer a valid test. The allowlist test is done in one of the filters.
Filters cannot return errors. To support that, it would introduce
additional overhead for very little value. If the need to verify
allowlists is so great, build primitive can be used.
Bug: 336807771
Change-Id: I15e4ab548dd985304daec8c1febcf241203980f2
diff --git a/Android.bp b/Android.bp
index 38481d9..dbac421 100644
--- a/Android.bp
+++ b/Android.bp
@@ -13607,7 +13607,6 @@
"src/trace_redaction/collect_frame_cookies.cc",
"src/trace_redaction/collect_system_info.cc",
"src/trace_redaction/collect_timeline_events.cc",
- "src/trace_redaction/filter_ftrace_using_allowlist.cc",
"src/trace_redaction/filter_packet_using_allowlist.cc",
"src/trace_redaction/find_package_uid.cc",
"src/trace_redaction/modify_process_trees.cc",
@@ -13615,15 +13614,12 @@
"src/trace_redaction/process_thread_timeline.cc",
"src/trace_redaction/proto_util.cc",
"src/trace_redaction/prune_package_list.cc",
- "src/trace_redaction/redact_ftrace_event.cc",
+ "src/trace_redaction/redact_ftrace_events.cc",
"src/trace_redaction/redact_process_events.cc",
"src/trace_redaction/redact_process_trees.cc",
"src/trace_redaction/redact_sched_events.cc",
- "src/trace_redaction/remap_scheduling_events.cc",
- "src/trace_redaction/scrub_ftrace_events.cc",
"src/trace_redaction/scrub_process_stats.cc",
"src/trace_redaction/scrub_trace_packet.cc",
- "src/trace_redaction/suspend_resume.cc",
"src/trace_redaction/trace_redaction_framework.cc",
"src/trace_redaction/trace_redactor.cc",
"src/trace_redaction/verify_integrity.cc",
@@ -13646,7 +13642,6 @@
"src/trace_redaction/prune_package_list_unittest.cc",
"src/trace_redaction/redact_process_events_unittest.cc",
"src/trace_redaction/redact_sched_events_unittest.cc",
- "src/trace_redaction/remap_scheduling_events_unittest.cc",
"src/trace_redaction/suspend_resume_unittest.cc",
"src/trace_redaction/verify_integrity_unittest.cc",
],
