Updates in network traffic annotations proto definition.

Updates in network traffic annotations proto definition.

Minor updates in comments, samples, and DEPS file for network traffic annotation proto.

BUG=

Review-Url: https://codereview.chromium.org/2691233002
Cr-Commit-Position: refs/heads/master@{#450292}
Committed: https://chromium.googlesource.com/chromium/src/+/1175597e824387e993a7dadd093ca58cdaae9246

[Ramin Halavati, 2017-02-14 07:32:43]

rhalavati@chromium.org changed reviewers:
+ battre@chromium.org, msramek@chromium.org

[Ramin Halavati, 2017-02-14 07:32:43]

Minor updates, please review.

[battre, 2017-02-14 08:09:17]

LGTM with a comment.

https://codereview.chromium.org/2691233002/diff/1/tools/traffic_annotation/sa...
File tools/traffic_annotation/sample_traffic_annotation.cc (right):

https://codereview.chromium.org/2691233002/diff/1/tools/traffic_annotation/sa...
tools/traffic_annotation/sample_traffic_annotation.cc:19: sender: "Spellcheck
Lookup"
I think that "Spellcheck Lookup" is an operation but not a sender. "Spellcheck"
or "Online Spellchecker" would be better.

[Ramin Halavati, 2017-02-14 08:12:13]

The CQ bit was checked by rhalavati@chromium.org

[Ramin Halavati, 2017-02-14 08:12:13]

The patchset sent to the CQ was uploaded after l-g-t-m from battre@chromium.org
Link to the patchset: https://codereview.chromium.org/2691233002/#ps20001
(title: "Comments addressed.")

[commit-bot: I haz the power, 2017-02-14 08:12:28]

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

[Ramin Halavati, 2017-02-14 08:12:40]

Comment addressed, landing.

https://codereview.chromium.org/2691233002/diff/1/tools/traffic_annotation/sa...
File tools/traffic_annotation/sample_traffic_annotation.cc (right):

https://codereview.chromium.org/2691233002/diff/1/tools/traffic_annotation/sa...
tools/traffic_annotation/sample_traffic_annotation.cc:19: sender: "Spellcheck
Lookup"
On 2017/02/14 08:09:17, battre wrote:
> I think that "Spellcheck Lookup" is an operation but not a sender.
"Spellcheck"
> or "Online Spellchecker" would be better.

Done.

[commit-bot: I haz the power, 2017-02-14 08:19:02]

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

[commit-bot: I haz the power, 2017-02-14 08:19:03]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Ramin Halavati, 2017-02-14 08:38:55]

The CQ bit was checked by rhalavati@chromium.org

[Ramin Halavati, 2017-02-14 08:38:55]

The patchset sent to the CQ was uploaded after l-g-t-m from battre@chromium.org
Link to the patchset: https://codereview.chromium.org/2691233002/#ps40001
(title: "DEPS updated.")

[commit-bot: I haz the power, 2017-02-14 08:39:15]

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-02-14 08:47:01]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487061535146750,
"parent_rev": "8f2a1baf12aa71d59d72e24583983b8a6b78340a", "commit_rev":
"1175597e824387e993a7dadd093ca58cdaae9246"}

[commit-bot: I haz the power, 2017-02-14 08:48:05]

Description was changed from

==========
Updates in network traffic annotations proto definition.

Minor updates in comments, samples, and DEPS file for network traffic annotation
proto.

BUG=
==========

to

==========
Updates in network traffic annotations proto definition.

Minor updates in comments, samples, and DEPS file for network traffic annotation
proto.

BUG=

Review-Url: https://codereview.chromium.org/2691233002
Cr-Commit-Position: refs/heads/master@{#450292}
Committed:
https://chromium.googlesource.com/chromium/src/+/1175597e824387e993a7dadd093c...
==========

[commit-bot: I haz the power, 2017-02-14 08:48:06]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/1175597e824387e993a7dadd093c...