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

Unified Diff: chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc

Issue 1530403003: Add anonymizer tool (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase to ToT Created 4 years, 11 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: chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc
diff --git a/chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc b/chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc
index 24aa8de1052eebab4e27c6479565435af24108d4..eed5ce53468a6bd286dabebda90eb1659a4c5a99 100644
--- a/chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc
+++ b/chrome/browser/feedback/system_logs/scrubbed_system_logs_fetcher.cc
@@ -36,7 +36,8 @@ ScrubbedSystemLogsFetcher::ScrubbedSystemLogsFetcher() {
data_sources_.push_back(new TouchLogSource());
// Debug Daemon data source - currently only this data source supports
- // the scrub_data parameter.
+ // the scrub_data parameter but all others get processed by Rewrite()
+ // as well.
const bool scrub_data = true;
data_sources_.push_back(new DebugDaemonLogSource(scrub_data));
#endif
@@ -47,4 +48,10 @@ ScrubbedSystemLogsFetcher::ScrubbedSystemLogsFetcher() {
ScrubbedSystemLogsFetcher::~ScrubbedSystemLogsFetcher() {
}
+void ScrubbedSystemLogsFetcher::Rewrite(const std::string& source_name,
+ SystemLogsResponse* response) {
+ for (auto& element : *response)
+ element.second = anonymizer_.Anonymize(element.second);
+}
+
} // namespace system_logs

Powered by Google App Engine
This is Rietveld 408576698