Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1349)

Unified Diff: rlz/lib/financial_ping.cc

Issue 2730263003: Network traffic annotation added to rlz/lib. (Closed)
Patch Set: Comments addressed. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: rlz/lib/financial_ping.cc
diff --git a/rlz/lib/financial_ping.cc b/rlz/lib/financial_ping.cc
index b9c200d5e9b6cb6c25b405e2547d9b790ac094f4..830fd2d641dd243a5ec532a7c115ba33d05e51aa 100644
--- a/rlz/lib/financial_ping.cc
+++ b/rlz/lib/financial_ping.cc
@@ -58,6 +58,7 @@ class InternetHandle {
#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"
@@ -318,8 +319,31 @@ bool FinancialPing::PingServer(const char* request, std::string* response) {
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 the effectiveness of a promotion. See the "
+ "Chrome Privacy Whitepaper for complete details."
+ trigger:
+ "1- At Chromium first run\.n"
+ "2- When Chromium is re-activated by a new promotion.\n"
+ "3- Once a week thereafter as long as Chromium is used.\n"
+ data:
+ "1- Non-unique cohort tag of when Chromium was installed.\n"
+ "2- Unique machine id on desktop platforms.\n"
+ "3- Whether Google is the default omnibox search.\n"
+ "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 |
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698