Chromium Code Reviews| 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..2da536515f867d742ead20b62a84f1e8199fa6c5 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,11 @@ 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* const anonymizer, |
|
jochen (gone - plz use gerrit)
2016/01/26 11:47:39
can this pointer also be const?
Polina Bondarenko
2016/01/26 12:47:30
No, AnonymizerTool can't be const, it calls non-co
|
| + const std::string& data); |
| private: |
| // Updates the system log upload enabled field from settings. |