Network traffic annotation added to safe_browsing

chrome/browser/safe_browsing/download_feedback.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/safe_browsing/download_feedback.h"
 
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
@@ skipping 110 matching lines @@
       ping_request_);
   DCHECK(r);
   r = report_metadata.mutable_download_response()->ParseFromString(
       ping_response_);
   DCHECK(r);
   file_size_ = report_metadata.download_request().length();
 
   std::string metadata_string;
   bool ok = report_metadata.SerializeToString(&metadata_string);
   DCHECK(ok);
+
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("...", R"(
+        semantics {
+          sender: "..."

[Nathan Parker, 2017/02/23 01:16:52]

"Safe Browsing Download Protection Feedback"

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          description: "..."

[Nathan Parker, 2017/02/23 01:16:53]

"When a user downloads a binary that Safe Browsing declares as suspicious,
opted-in clients may upload that binary to Safe Browsing to improve the
classification.  This helps protect users from malware and unwanted software."

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          trigger: "..."

[Nathan Parker, 2017/02/23 01:16:52]

"When a download-protection verdict is !SAFE, and the server requests the binary
be uploaded, and the user is opted in to extended-reporting, the browser will
upload the binary to Google."

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          data: "..."
+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER
+        }
+        policy {
+          cookies_allowed: false/true

[Nathan Parker, 2017/02/23 01:16:53]

true

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          cookies_store: "..."

[Nathan Parker, 2017/02/23 01:16:53]

Safe Browsing cookie store

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          setting: "..."

[Nathan Parker, 2017/02/23 01:16:53]

I'm not sure where we say that this is enabled by the user checking the box
"Automatically report details of possible security incidents to Google," and
that there's a policy where the admin can prevent that box from getting checked.

[Ramin Halavati, 2017/02/23 08:18:24]

Done.

+          policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              value: ...
+            }
+          }
+          policy_exception_justification: "..."
+        })");
+
   uploader_ = TwoPhaseUploader::Create(
       request_context_getter_.get(), file_task_runner_.get(),
       GURL(kSbFeedbackURL), metadata_string, file_path_,
       TwoPhaseUploader::ProgressCallback(),
       base::Bind(&DownloadFeedbackImpl::FinishedUpload, base::Unretained(this),
-                 finish_callback));
+                 finish_callback),
+      traffic_annotation);
   uploader_->Start();
 }
 
 void DownloadFeedbackImpl::FinishedUpload(base::Closure finish_callback,
                                           TwoPhaseUploader::State state,
                                           int net_error,
                                           int response_code,
                                           const std::string& response_data) {
   DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
   DVLOG(1) << __func__ << " " << state << " rlen=" << response_data.size();
@@ skipping 64 matching lines @@
         new DownloadFeedbackImpl(request_context_getter, file_task_runner,
                                  file_path, ping_request, ping_response));
   }
   return DownloadFeedback::factory_->CreateDownloadFeedback(
         request_context_getter, file_task_runner, file_path, ping_request,
         ping_response);
 }
 
 }  // namespace safe_browsing