Network traffic annotation added to image_data_fetcher.

chrome/browser/search/thumbnail_source.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.
 
 #include "chrome/browser/search/thumbnail_source.h"
 
 #include "base/callback.h"
 #include "base/memory/ref_counted_memory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/instant_io_context.h"
 #include "chrome/browser/thumbnails/thumbnail_service.h"
 #include "chrome/browser/thumbnails/thumbnail_service_factory.h"
 #include "chrome/common/url_constants.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_request.h"
 #include "url/gurl.h"
 
+namespace {
+
+constexpr net::NetworkTrafficAnnotationTag kTrafficAnnotation =
+    net::DefineNetworkTrafficAnnotation("...", R"(

[Marc Treib, 2017/05/09 09:43:47]

thumbnail_source

[Ramin Halavati, 2017/05/09 11:08:24]

Done.

+      semantics {
+        sender: "..."

[Marc Treib, 2017/05/09 09:43:47]

ThumbnailSource

[Ramin Halavati, 2017/05/09 11:08:24]

Done.

+        description: "..."

[Marc Treib, 2017/05/09 09:43:47]

Retrieves thumbnails for site suggestions based on the user's synced browsing
history, for use e.g. on the New Tab page.

[Ramin Halavati, 2017/05/09 11:08:24]

Done.

+        trigger: "..."

[Marc Treib, 2017/05/09 09:43:47]

Triggered when a thumbnail for a suggestion is required (e.g. on the New Tab
page), and no local thumbnail is available.

[Ramin Halavati, 2017/05/09 11:08:24]

Done.

+        data: "..."

[Marc Treib, 2017/05/09 09:43:47]

None.

[Ramin Halavati, 2017/05/09 11:08:24]

Doesn't it sent the URL to that thumbnail? Isn't it related to user history?

[Marc Treib, 2017/05/09 11:38:34]

Right, the request URL includes the URL of the page for which we want a
thumbnail as a parameter. It looks like this:
https://www.google.com/webpagethumbnail?c=63&d=https://cs.chromium.org/search....

So how about:
The URL for which to retrieve a thumbnail.

[Ramin Halavati, 2017/05/09 13:16:47]

Done.

+        destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL

[Ramin Halavati, 2017/05/09 11:08:24]

GOOGLE_OWNED_SERVICE?

[Marc Treib, 2017/05/09 11:38:34]

Yes. Sorry, missed that one.

[Ramin Halavati, 2017/05/09 13:16:47]

Done.

+      }
+      policy {
+        cookies_allowed: false/true

[Marc Treib, 2017/05/09 09:43:47]

false

[Ramin Halavati, 2017/05/09 11:08:24]

Done.

+        cookies_store: "..."
+        setting: "..."

[Marc Treib, 2017/05/09 09:43:47]

Same here: Cannot be disabled individually, but users can disable the whole
SuggestionsService, see here:
https://cs.chromium.org/chromium/src/components/suggestions/suggestions_servi...

[Ramin Halavati, 2017/05/09 13:16:47]

Done.

+        chrome_policy {
+          [POLICY_NAME] {
+            policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+            [POLICY_NAME]: ... //(value to disable it)
+          }
+        }
+        policy_exception_justification: "..."
+      })");
+
+}  // namespace
+
 // The delimiter between the first url and the fallback url passed to
 // StartDataRequest.
 const char kUrlDelimiter[] = "?fb=";
 
 // Set ThumbnailService now as Profile isn't thread safe.
 ThumbnailSource::ThumbnailSource(Profile* profile, bool capture_thumbnails)
     : thumbnail_service_(ThumbnailServiceFactory::GetForProfile(profile)),
       capture_thumbnails_(capture_thumbnails),
-      image_data_fetcher_(profile->GetRequestContext()),
+      image_data_fetcher_(profile->GetRequestContext(), kTrafficAnnotation),
       weak_ptr_factory_(this) {}
 
 ThumbnailSource::~ThumbnailSource() = default;
 
 std::string ThumbnailSource::GetSource() const {
   return capture_thumbnails_ ?
       chrome::kChromeUIThumbnailHost2 : chrome::kChromeUIThumbnailHost;
 }
 
 void ThumbnailSource::StartDataRequest(
@@ skipping 80 matching lines @@
     const content::URLDataSource::GotDataCallback& callback,
     const std::string& image_data,
     const image_fetcher::RequestMetadata& metadata) {
   if (image_data.empty()) {
     callback.Run(nullptr);
     return;
   }
   std::string image_data_copy = image_data;
   callback.Run(base::RefCountedString::TakeString(&image_data_copy));
 }