Specify TaskType of posted Task explicitly in networking classes (5)

Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696
Committed: https://crrev.com/c7d47a47d6bd59baa0197569991deb7958d8657e
Cr-Commit-Position: refs/heads/master@{#439385}

[tzik, 2016-12-18 13:51:40]

Description was changed from

==========
Specify TaskType of posted Task explicitly in networking classes (5)

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

to

==========
Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696
==========

[tzik, 2016-12-18 13:51:51]

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

[commit-bot: I haz the power, 2016-12-18 13:51:57]

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

[tzik, 2016-12-18 14:40:34]

tzik@chromium.org changed reviewers:
+ haraken@chromium.org

[tzik, 2016-12-18 14:40:35]

PTAL

[haraken, 2016-12-18 14:53:20]

LGTM

[commit-bot: I haz the power, 2016-12-18 15:21:29]

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

[commit-bot: I haz the power, 2016-12-18 15:21:30]

Dry run: This issue passed the CQ dry run.

[tzik, 2016-12-18 22:52:17]

The CQ bit was checked by tzik@chromium.org

[commit-bot: I haz the power, 2016-12-18 22:52:29]

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, 2016-12-18 22:56:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482101537663390, "parent_rev":
"fa7f02ecc0bb56e945c1e73ad9ebcf6ffefce886", "commit_rev":
"394081dabdb992f02c332416d341a485362840f7"}

[commit-bot: I haz the power, 2016-12-18 22:56:33]

Description was changed from

==========
Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696
==========

to

==========
Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696

Review-Url: https://codereview.chromium.org/2584253002
==========

[commit-bot: I haz the power, 2016-12-18 22:56:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-18 22:59:33]

Description was changed from

==========
Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696

Review-Url: https://codereview.chromium.org/2584253002
==========

to

==========
Specify TaskType of posted Task explicitly in networking classes

This CL assigns TaskType to ContentSecurityPolicy::reportViolation
and NetworkStateNotifier::notifyObservers.

For CSP, the spec doesn't explicitly specify the task source. However,
since it's a fetch request, TaskType::Networking should be the best fit.
http://wicg.github.io/reporting/#try-delivery

For NetworkStateNotifier, the spec specifies it should be TaskType::Networking.
https://wicg.github.io/netinfo/#dependencies-and-definitions

BUG=624696

Committed: https://crrev.com/c7d47a47d6bd59baa0197569991deb7958d8657e
Cr-Commit-Position: refs/heads/master@{#439385}
==========

[commit-bot: I haz the power, 2016-12-18 22:59:34]

Patchset 1 (id:??) landed as
https://crrev.com/c7d47a47d6bd59baa0197569991deb7958d8657e
Cr-Commit-Position: refs/heads/master@{#439385}