Network traffic annotation added to rlz/lib.

rlz/lib/financial_ping.cc

 // Copyright (c) 2012 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.
 //
 // Library functions related to the Financial Server ping.
 
 #include "rlz/lib/financial_ping.h"
 
 #include <stdint.h>
 
@@ skipping 40 matching lines @@
 
 }  // namespace
 
 #else
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "base/time/time.h"
 #include "net/base/load_flags.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_fetcher.h"
 #include "net/url_request/url_fetcher_delegate.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "url/gurl.h"
 
 #endif
 
 namespace {
 
@@ skipping 240 matching lines @@
   // Ensure that we have a MessageLoop.
   if (!base::MessageLoop::current())
     message_loop.reset(new base::MessageLoop);
   base::RunLoop loop;
   FinancialPingUrlFetcherDelegate delegate(loop.QuitClosure());
 
   std::string url = base::StringPrintf("http://%s:%d%s",
                                        kFinancialServer, kFinancialPort,
                                        request);
 
-  std::unique_ptr<net::URLFetcher> fetcher =
-      net::URLFetcher::Create(GURL(url), net::URLFetcher::GET, &delegate);
+  net::NetworkTrafficAnnotationTag traffic_annotation =
+      net::DefineNetworkTrafficAnnotation("rlz_ping", R"(
+        semantics {
+          sender: "RLZ Ping"
+          description:
+            "Used for measuring effectiveness of a promotion. See Chromium "

[msramek, 2017/03/07 10:24:49]

nit: Not a native speaker, but shouldn't this be "the effectiveness"?

[Ramin Halavati, 2017/03/07 12:46:00]

Done.

+            "privacy policy for complete details: https://www.google.com/chrome"

[msramek, 2017/03/07 10:24:49]

"See the Chrome Privacy Whitepaper"

[Ramin Halavati, 2017/03/07 12:46:00]

Done.

+            "/browser/privacy/whitepaper.html#measurepromotions"
+          trigger:
+            "1/ At Chromium first run, "

[msramek, 2017/03/07 10:24:49]

Should we add "\n" so it looks good when strings are extracted?

[Ramin Halavati, 2017/03/07 12:46:00]

Done.

+            "2/ When Chromium is re-activated by a new promotion, "
+            "3/ Once a week thereafter as long as Chromium is used."
+          data:
+            "1/ Non-unique cohort tag of when Chromium was installed, "
+            "2/ Unique machine id on desktop platforms, "
+            "3/ Whether Google is the default omnibox search, "
+            "4/ Whether google.com is the default home page."
+          destination: GOOGLE_OWNED_SERVICE
+        }
+        policy {
+          cookies_allowed: false
+          setting: "This feature cannot be disabled in settings."
+          policy_exception_justification: "Not implemented."
+        })");
+  std::unique_ptr<net::URLFetcher> fetcher = net::URLFetcher::Create(
+      GURL(url), net::URLFetcher::GET, &delegate, traffic_annotation);
 
   fetcher->SetLoadFlags(net::LOAD_DISABLE_CACHE |
                         net::LOAD_DO_NOT_SEND_AUTH_DATA |
                         net::LOAD_DO_NOT_SEND_COOKIES |
                         net::LOAD_DO_NOT_SAVE_COOKIES);
 
   // Ensure rlz_lib::SetURLRequestContext() has been called before sending
   // pings.
   fetcher->SetRequestContext(context);
 
@@ skipping 74 matching lines @@
 }
 
 bool WasSendFinancialPingInterrupted() {
   return send_financial_ping_interrupted_for_test;
 }
 
 }  // namespace test
 #endif
 
 }  // namespace rlz_lib