Network traffic annotation added to indexed_db.

content/browser/indexed_db/indexed_db_backing_store.cc

Index: content/browser/indexed_db/indexed_db_backing_store.cc
diff --git a/content/browser/indexed_db/indexed_db_backing_store.cc b/content/browser/indexed_db/indexed_db_backing_store.cc
index 9d2b89ba3da8e40722b80b9176e1192d59555999..dcbd66dfb725f17b5609fdfd435e05127bc6737c 100644
--- a/content/browser/indexed_db/indexed_db_backing_store.cc
+++ b/content/browser/indexed_db/indexed_db_backing_store.cc
@@ -39,6 +39,7 @@
 #include "content/common/indexed_db/indexed_db_key_path.h"
 #include "content/common/indexed_db/indexed_db_key_range.h"
 #include "content/public/browser/browser_thread.h"
+#include "net/traffic_annotation/network_traffic_annotation.h"
 #include "net/url_request/url_request_context.h"
 #include "storage/browser/blob/blob_data_handle.h"
 #include "storage/browser/fileapi/file_stream_writer.h"
@@ -2556,9 +2557,30 @@ class LocalWriteClosure : public FileWriterDelegate::DelegateWriteCallback,
     DCHECK(blob_url.is_valid());
     net::URLRequestContext* request_context =
         request_context_getter->GetURLRequestContext();
+    net::NetworkTrafficAnnotationTag traffic_annotation =

[jsbell, 2017/05/03 15:51:56]

The request is for a content of a local file ("blob") resource. This will not
actually hit the network.

The code is copying data already held in Chrome - referenced by a blob URL -
into database storage.

[Ramin Halavati, 2017/05/04 04:41:01]

Thank you for clarification. As a presubmit checker will be added for all
network requests that are in codes which are compiled during building Chrome, we
need to annotate this as well. But we can choose destination as LOCAL and don't
need policy.

+        net::DefineNetworkTrafficAnnotation("...", R"(

[jsbell, 2017/05/04 15:53:16]

persist_blob_to_indexeddb

[Ramin Halavati, 2017/05/05 08:43:21]

Done.

+        semantics {
+          sender: "..."

[jsbell, 2017/05/04 15:53:16]

Indexed DB

[Ramin Halavati, 2017/05/05 08:43:22]

Done.

+          description: "..."

[jsbell, 2017/05/04 15:53:16]

A web page's script has created a Blob (or File) object (either directly via
constructors, or by using file upload to a form, or via a fetch()). The script
has then made a request to store data including the Blob via the Indexed DB API.
As part of committing the database transaction, the content of the Blob is being
copied into a file in the database's directory.

[Ramin Halavati, 2017/05/05 08:43:21]

Done.

+          trigger: "..."

[jsbell, 2017/05/04 15:53:16]

The script has then made a request to store data including a Blob via the
Indexed DB API.

[Ramin Halavati, 2017/05/05 08:43:21]

Done.

+          data: "..."

[jsbell, 2017/05/04 15:53:17]

A Blob or File object referenced by script, either created directly via
constructors, or by using file upload to a form, or drag/dop, or via a fetch()
or other APIs that produce Blobs.

[Ramin Halavati, 2017/05/05 08:43:21]

Done.

+          destination: WEBSITE/GOOGLE_OWNED_SERVICE/OTHER/LOCAL

[jsbell, 2017/05/04 15:53:16]

LOCAL

[Ramin Halavati, 2017/05/05 08:43:21]

Done.

+        }
+        policy {
+          cookies_allowed: false/true

[Ramin Halavati, 2017/05/05 08:43:22]

Cookies are enabled by default, it seems that you are not using them, can I add
a CL that adds net::LOAD_DO_NOT_SAVE_COOKIES and                         
net::LOAD_DO_NOT_SEND_COOKIES load flags?

[jsbell, 2017/05/05 18:23:50]

Yes please!

[Ramin Halavati, 2017/05/08 09:51:04]

Done, in CL https://codereview.chromium.org/2864243002

+          cookies_store: "..."
+          setting: "..."
+          chrome_policy {
+            [POLICY_NAME] {
+              policy_options {mode: MANDATORY/RECOMMENDED/UNSET}
+              [POLICY_NAME]: ... //(value to disable it)
+            }
+          }
+          policy_exception_justification: "..."
+        })");
     std::unique_ptr<net::URLRequest> blob_request(
         request_context->CreateRequest(blob_url, net::DEFAULT_PRIORITY,
-                                       delegate.get()));
+                                       delegate.get(), traffic_annotation));
 
     this->file_path_ = file_path;
     this->last_modified_ = last_modified;