Network traffic annotation stored in url_request.

Network traffic annotation stored in url_request.

Network traffic annotation tags are stored in url_request.

BUG=656607
Review-Url: https://codereview.chromium.org/2890293005
Cr-Commit-Position: refs/heads/master@{#473874}
Committed: https://chromium.googlesource.com/chromium/src/+/0e2a2278827bd1ef68423438211030a22b60e331

[Ramin Halavati, 2017-05-19 16:48:49]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-19 16:49:20]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-19 17:00:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-19 17:00:11]

Dry run: Try jobs failed on following builders:
  android_cronet on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)

[Ramin Halavati, 2017-05-22 05:45:50]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 05:46:02]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-22 05:56:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-22 05:56:50]

Dry run: Try jobs failed on following builders:
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Ramin Halavati, 2017-05-22 06:28:51]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 06:29:07]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Ramin Halavati, 2017-05-22 06:29:25]

Patchset #2 (id:20001) has been deleted

[Ramin Halavati, 2017-05-22 06:29:55]

Patchset #2 (id:40001) has been deleted

[Ramin Halavati, 2017-05-22 06:30:05]

Patchset #2 (id:60001) has been deleted

[Ramin Halavati, 2017-05-22 06:30:12]

Patchset #2 (id:80001) has been deleted

[commit-bot: I haz the power, 2017-05-22 07:06:45]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-22 07:06:46]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ramin Halavati, 2017-05-22 07:52:31]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 07:52:51]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-22 08:37:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-22 08:37:38]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ramin Halavati, 2017-05-22 08:42:19]

The CQ bit was checked by rhalavati@chromium.org to run a CQ dry run

[Ramin Halavati, 2017-05-22 08:42:27]

Patchset #2 (id:100001) has been deleted

[Ramin Halavati, 2017-05-22 08:42:34]

Patchset #2 (id:120001) has been deleted

[commit-bot: I haz the power, 2017-05-22 08:42:45]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-22 10:05:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-22 10:05:27]

Dry run: This issue passed the CQ dry run.

[Ramin Halavati, 2017-05-22 10:43:24]

Patchset #1 (id:1) has been deleted

[Ramin Halavati, 2017-05-22 14:26:47]

rhalavati@chromium.org changed reviewers:
+ xunjieli@chromium.org

[Ramin Halavati, 2017-05-22 14:26:48]

Helen,

This CL adds unique id of network traffic annotations to URLRequest.
Please review.

[xunjieli, 2017-05-22 14:34:11]

On 2017/05/22 14:26:48, Ramin Halavati wrote:
> Helen,
> 
> This CL adds unique id of network traffic annotations to URLRequest.
> Please review.

net/ LGTM. Thanks.

[Ramin Halavati, 2017-05-23 04:47:14]

rhalavati@chromium.org changed reviewers:
+ jochen@chromium.org

[Ramin Halavati, 2017-05-23 04:47:15]

Thank you Helen.

Jochen,
This CL stores network annotation tag in URLRequest with would be used to get
stats on network usage and crashes. The changes outside the net folder are all
reflecting this change in test files. Could you please give a high level
approval, or you prefer if I add all respective owners?

[jochen (gone - plz use gerrit), 2017-05-23 11:14:41]

lgtm

[Ramin Halavati, 2017-05-23 11:32:54]

The CQ bit was checked by rhalavati@chromium.org

[commit-bot: I haz the power, 2017-05-23 11:33:11]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-23 13:10:35]

CQ is committing da patch.

Bot data: {"patchset_id": 120002, "attempt_start_ts": 1495539174516640,
"parent_rev": "f04e36c1450b210d10cbffb115c72c6676b58a8e", "commit_rev":
"0e2a2278827bd1ef68423438211030a22b60e331"}

[commit-bot: I haz the power, 2017-05-23 13:10:47]

Description was changed from

==========
Network traffic annotation stored in url_request.

Network traffic annotation tags are stored in url_request.

BUG=656607
==========

to

==========
Network traffic annotation stored in url_request.

Network traffic annotation tags are stored in url_request.

BUG=656607

Review-Url: https://codereview.chromium.org/2890293005
Cr-Commit-Position: refs/heads/master@{#473874}
Committed:
https://chromium.googlesource.com/chromium/src/+/0e2a2278827bd1ef684234382110...
==========

[commit-bot: I haz the power, 2017-05-23 13:10:48]

Committed patchset #1 (id:120002) as
https://chromium.googlesource.com/chromium/src/+/0e2a2278827bd1ef684234382110...