Add anonymizer tool

components/feedback/feedback_common.h

Index: components/feedback/feedback_common.h
diff --git a/components/feedback/feedback_common.h b/components/feedback/feedback_common.h
index a95301d5c60d45aaafcc9a34f08751d7305c707b..1b3e65819b59c51fb2c5c51fd63a4735a20cae05 100644
--- a/components/feedback/feedback_common.h
+++ b/components/feedback/feedback_common.h
@@ -48,8 +48,11 @@ class FeedbackCommon : public base::RefCountedThreadSafe<FeedbackCommon> {
   void CompressFile(const base::FilePath& filename,
                     const std::string& zipname,
                     scoped_ptr<std::string> data);
+  // File attachments are not anonymized. It is the responsibility of the caller
+  // to remove personally identifying information by using the AnonymizerTool.
   void AddFile(const std::string& filename, scoped_ptr<std::string> data);
 
+  // Logs are anonymized via the AnonymizerTool.
   void AddLog(const std::string& name, const std::string& value);
   void AddLogs(scoped_ptr<SystemLogsMap> logs);
   void CompressLogs();
@@ -59,6 +62,8 @@ class FeedbackCommon : public base::RefCountedThreadSafe<FeedbackCommon> {
 
   // Fill in |feedback_data| with all the data that we have collected.
   // CompressLogs() must have already been called.
+  // File attachments in the feedback data need to be properly anonymized before

[vasilii, 2015/12/17 12:40:37]

The comment above says that the file attachments aren't anonymised.

[battre, 2015/12/17 13:24:24]

I don't understand. Neither AddFile nor PrepareReport takes care of anonymizing
file attachments. This needs to be done be the caller.

+  // reporting using the AnonymizerTool.
   void PrepareReport(userfeedback::ExtensionSubmit* feedback_data) const;
 
   // Getters