Added anonymizer to SystemLogUploader.

chrome/browser/chromeos/policy/system_log_uploader.h

Index: chrome/browser/chromeos/policy/system_log_uploader.h
diff --git a/chrome/browser/chromeos/policy/system_log_uploader.h b/chrome/browser/chromeos/policy/system_log_uploader.h
index 5a3d6243c12737285c12f1ce5d68f0f322f0ea77..48683a0fcf6b03911a46ceddc4953799efac9fd3 100644
--- a/chrome/browser/chromeos/policy/system_log_uploader.h
+++ b/chrome/browser/chromeos/policy/system_log_uploader.h
@@ -21,6 +21,10 @@ namespace base {
 class SequencedTaskRunner;
 }
 
+namespace feedback {
+class AnonymizerTool;
+}
+
 namespace policy {
 
 // Class responsible for periodically uploading system logs, it handles the
@@ -78,9 +82,10 @@ class SystemLogUploader : public UploadJob::Delegate {
   void OnSuccess() override;
   void OnFailure(UploadJob::ErrorCode error_code) override;
 
-  // Remove lines from |data| that contain common PII (IP addresses, SSIDs, URLs
-  // e-mail addresses).
-  static std::string RemoveSensitiveData(const std::string& data);
+  // Remove lines from |data| that contain common PII (IP addresses, BSSIDs,
+  // SSIDs, URLs, e-mail addresses).
+  static std::string RemoveSensitiveData(feedback::AnonymizerTool& anonymizer,

[Andrew T Wilson (Slow), 2016/01/25 11:06:42]

If you're passing in an anonymizer by reference, then it needs to be const.

[Polina Bondarenko, 2016/01/25 15:08:33]

Fixed to const ptr.

The object can't be const, because the anonymizer is changed in process of
anonymizing logs. E.g., it lists all IP addresses to count them.

+                                         const std::string& data);
 
  private:
   // Updates the system log upload enabled field from settings.