Network traffic annotation added to content_hash_fetcher.cc.

extensions/browser/content_hash_fetcher.cc

 // Copyright 2014 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 "extensions/browser/content_hash_fetcher.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <memory>
@@ skipping 13 matching lines @@
 #include "content/public/browser/browser_thread.h"
 #include "crypto/sha2.h"
 #include "extensions/browser/computed_hashes.h"
 #include "extensions/browser/content_hash_tree.h"
 #include "extensions/browser/content_verifier_delegate.h"
 #include "extensions/browser/verified_contents.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/file_util.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_status.h"
 
 namespace {
 
 typedef std::set<base::FilePath> SortedFilePathSet;
 
 }  // namespace
 
@@ skipping 189 matching lines @@
 
 void ContentHashFetcherJob::DoneCheckingForVerifiedContents(bool found) {
   if (IsCancelled())
     return;
   if (found) {
     VLOG(1) << "Found verified contents for " << extension_id_;
     DoneFetchingVerifiedContents(true);
   } else {
     VLOG(1) << "Missing verified contents for " << extension_id_
             << ", fetching...";
-    url_fetcher_ =
-        net::URLFetcher::Create(fetch_url_, net::URLFetcher::GET, this);
+    net::NetworkTrafficAnnotationTag traffic_annotation =
+        net::DefineNetworkTrafficAnnotation("...", R"(

[Devlin, 2017/03/11 03:17:13]

ContentHashVerificationJob

[Ramin Halavati, 2017/03/11 16:04:59]

Done.

+          semantics {
+            sender: "..."

[Devlin, 2017/03/11 03:17:13]

ContentHashFetcherJob

[Ramin Halavati, 2017/03/11 16:05:00]

Done.

+            description: "..."

[Devlin, 2017/03/11 03:17:13]

The request sent to retrieve the verified_contents.json file for an extension
id.

[Ramin Halavati, 2017/03/11 16:05:00]

Done.

+            trigger: "..."

[Devlin, 2017/03/11 03:17:13]

An extension from the webstore is missing the verified_contents.json file
required for extension content verification.

[Ramin Halavati, 2017/03/11 16:05:00]

Done.

+            data: "..."

[Devlin, 2017/03/11 03:17:13]

The extension id and extension version.

[Ramin Halavati, 2017/03/11 16:04:59]

Done.

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

[Devlin, 2017/03/11 03:17:13]

This feature cannot be directly disabled; it is enabled if any extension from
the webstore is installed in the browser.

[Ramin Halavati, 2017/03/11 16:05:00]

Done.

+            policy {
+              [POLICY_NAME] {
+                policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+                [POLICY_NAME]: ... //(value to disable it)
+              }
+            }
+            policy_exception_justification: "..."

[Devlin, 2017/03/11 03:17:13]

This policy only takes effect if the user has extensions from the webstore; if
the user has extensions from the webstore, this feature is required to ensure
the extensions match what is distributed by the store.

[Ramin Halavati, 2017/03/11 16:04:59]

Done.

+          })");
+    url_fetcher_ = net::URLFetcher::Create(fetch_url_, net::URLFetcher::GET,
+                                           this, traffic_annotation);
     url_fetcher_->SetRequestContext(request_context_);
     url_fetcher_->SetLoadFlags(net::LOAD_DO_NOT_SEND_COOKIES |
                                net::LOAD_DO_NOT_SAVE_COOKIES |
                                net::LOAD_DISABLE_CACHE);
     url_fetcher_->SetAutomaticallyRetryOnNetworkChanges(3);
     url_fetcher_->Start();
   }
 }
 
 // Helper function to let us pass ownership of a string via base::Bind with the
@@ skipping 242 matching lines @@
 
   for (JobMap::iterator i = jobs_.begin(); i != jobs_.end(); ++i) {
     if (i->second.get() == job) {
       jobs_.erase(i);
       break;
     }
   }
 }
 
 }  // namespace extensions