Network traffic annotation added to chrome::BitmapFetcher.

chrome/browser/extensions/webstore_install_helper.cc

Index: chrome/browser/extensions/webstore_install_helper.cc
diff --git a/chrome/browser/extensions/webstore_install_helper.cc b/chrome/browser/extensions/webstore_install_helper.cc
index 691ec84901659706c6e3008c225f0e0a39aa91f8..8e6cea468e83d153103c15797509968b115356b4 100644
--- a/chrome/browser/extensions/webstore_install_helper.cc
+++ b/chrome/browser/extensions/webstore_install_helper.cc
@@ -10,6 +10,7 @@
 #include "components/safe_json/safe_json_parser.h"
 #include "content/public/browser/browser_thread.h"
 #include "net/base/load_flags.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_request.h"
 
 using content::BrowserThread;
@@ -53,7 +54,36 @@ void WebstoreInstallHelper::Start() {
     // No existing |icon_fetcher_| to avoid unbalanced AddRef().
     CHECK(!icon_fetcher_.get());
     AddRef();  // Balanced in OnFetchComplete().
-    icon_fetcher_.reset(new chrome::BitmapFetcher(icon_url_, this));
+
+    net::NetworkTrafficAnnotationTag traffic_annotation =
+        net::DefineNetworkTrafficAnnotation("webstore_install_helper", R"(
+          semantics {
+            sender: "Webstore Install Helper"
+            description:
+              "Fetches the bitmap corresponding to an extension icon."
+            trigger:
+              "This can happen in a few different circumstances: "
+              "1-User initiated an install from the Chrome Web Store."
+              "2-User initiated an inline installation from another website."
+              "3-Loading of kiosk app data on Chrome OS (provided that the "
+              "kiosk app is a Web Store app)."
+            data:
+              "The url of the icon for the extension, which includes the "
+              "extension id."
+            destination: GOOGLE_OWNED_SERVICE
+          }
+          policy {
+            cookies_allowed: false
+            setting:
+              "There's no direct Chrome setting to disable this, but you could "
+              "uninstall all extensions and not install (or begin the "
+              "installation flow for) any more."
+            policy_exception_justification:
+              "Not implemented, considered not useful."

[Devlin, 2017/02/16 18:00:50]

or similar to the avatar case, "No content is being uploaded or saved; this
request merely downloads a publicly available PNG file."

[Ramin Halavati, 2017/02/17 06:30:12]

I think this is implied in |data| part and it's sufficient to say it's not
useful here.

+          })");
+
+    icon_fetcher_.reset(
+        new chrome::BitmapFetcher(icon_url_, this, traffic_annotation));
     icon_fetcher_->Init(
         context_getter_, std::string(),
         net::URLRequest::CLEAR_REFERRER_ON_TRANSITION_FROM_SECURE_TO_INSECURE,