Network traffic annotation added to feedback_uploader_chrome.

components/feedback/feedback_uploader_chrome.cc

Index: components/feedback/feedback_uploader_chrome.cc
diff --git a/components/feedback/feedback_uploader_chrome.cc b/components/feedback/feedback_uploader_chrome.cc
index 0b18305d920b12584508bebbf8cb9d4b63cfb666..51223ab59c38e4275a9a5aba5110de4bd50f4583 100644
--- a/components/feedback/feedback_uploader_chrome.cc
+++ b/components/feedback/feedback_uploader_chrome.cc
@@ -20,6 +20,7 @@
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/storage_partition.h"
 #include "net/base/load_flags.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 #include "url/gurl.h"
 
@@ -47,14 +48,33 @@ FeedbackUploaderChrome::FeedbackUploaderChrome(
 void FeedbackUploaderChrome::DispatchReport(const std::string& data) {
   GURL post_url(url_);
 
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("chrome_feedback_report_app", R"(
+        semantics {
+          sender: "Chrome Feedback Report App"
+          description:
+            "This uploads the feedback report to Google feedback server."

[Ramin Halavati, 2017/03/11 09:42:54]

May be it would be good if you describe what is the feedback about. This text
should be clear enough so that a non-very-technical person finds out why this
data is sent.

[afakhry, 2017/03/14 01:36:42]

I'm not sure how to describe it in a simple way. This is the feedback report
sent by the user when he presses Alt+Shift+i to report a bug or a feedback in
general. Along with it is sent system logs that helps us to diagnose the issue.

[Ramin Halavati, 2017/03/14 06:31:57]

Thank you, I think this description is quite good. I updated the text based on
this.

+          trigger: "When the feedback report is being uploaded."

[Ramin Halavati, 2017/03/11 09:42:54]

Is user asked for sending feedback? In which cases feedback is generated?

[afakhry, 2017/03/14 01:36:42]

This is a user-generated event, as a result of pressing Alt+Shift+i. See
https://support.google.com/chrome/answer/95315?hl=en

[Ramin Halavati, 2017/03/14 06:31:57]

Done.

+          data:
+            "Contents of the report, user entered message, and useful "

[Ramin Halavati, 2017/03/11 09:42:54]

Can you add details on content items? Is there any user specific data?

[afakhry, 2017/03/14 01:36:42]

We anonymize the logs to remove any user-private data. The user can view the
system information before sending, and can choose to send the feedback report
without the logs, the screenshot, or even his email address.

[Ramin Halavati, 2017/03/14 06:31:57]

Done.

+            "debugging logs."
+          destination: GOOGLE_OWNED_SERVICE
+        }
+        policy {
+          cookies_allowed: false
+          setting: "..."

[Ramin Halavati, 2017/03/11 09:42:54]

I didn't find any settings disabling feedback, is there? Or can we add this
feedback is sent only by direct approval of the user.?

[afakhry, 2017/03/14 01:36:42]

I'm not aware of any setting to disable feedback.
Yes, we can add that.
I vaguely remember that in kiosk devices, some feedback reports are sent
periodically in the background, but I think for that to happen the user has to
be opt-in. +rkc@ who might know more about this.

[Ramin Halavati, 2017/03/14 06:31:57]

Acknowledged.

+          policy_exception_justification: "Not implemented."
+        })");
   // Note: FeedbackUploaderDelegate deletes itself and the fetcher.
   net::URLFetcher* fetcher =
       net::URLFetcher::Create(
           post_url, net::URLFetcher::POST,
           new FeedbackUploaderDelegate(
-              data, base::Bind(&FeedbackUploaderChrome::UpdateUploadTimer,
-                               AsWeakPtr()),
-              base::Bind(&FeedbackUploaderChrome::RetryReport, AsWeakPtr())))
+              data,
+              base::Bind(&FeedbackUploaderChrome::UpdateUploadTimer,
+                         AsWeakPtr()),
+              base::Bind(&FeedbackUploaderChrome::RetryReport, AsWeakPtr())),
+          traffic_annotation)
           .release();
   data_use_measurement::DataUseUserData::AttachToFetcher(
       fetcher, data_use_measurement::DataUseUserData::FEEDBACK_UPLOADER);