Reporting: Plumb into UrlRequest{HttpJob,Context}.

Reporting: Plumb into UrlRequest{HttpJob,Context}.

Reporting is a spec for delivering out-of-band reports from various
other parts of the browser. See http://wicg.github.io/reporting/ for
the spec, or https://goo.gl/pygX5I for details of the planned
implementation in Chromium.

This CL adds a member in URLRequestContext{,Storage} for the
ReportingService and calls ProcessHeader from URLRequestHttpJob when
a Report-To header is received under the right circumstances.

BUG=704259
Review-Url: https://codereview.chromium.org/2769703006
Cr-Commit-Position: refs/heads/master@{#464155}
Committed: https://chromium.googlesource.com/chromium/src/+/2b8dd2d1d8c2c17e5715853ad74cf2891b3c87c7

[Julia Tuttle, 2017-03-24 16:42:02]

juliatuttle@chromium.org changed reviewers:
+ shivanisha@chromium.org

[Julia Tuttle, 2017-03-24 16:42:03]

PTAL, shivanisha.

[shivanisha, 2017-03-24 18:47:15]

On 2017/03/24 at 16:42:03, juliatuttle wrote:
> PTAL, shivanisha.

Looking good % the ssl todo that remains.

[Julia Tuttle, 2017-03-27 15:09:29]

PTAL, shivanisha.

[Julia Tuttle, 2017-03-31 16:26:20]

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

[commit-bot: I haz the power, 2017-03-31 16:27:43]

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-03-31 16:33:39]

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

[commit-bot: I haz the power, 2017-03-31 16:33:40]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Julia Tuttle, 2017-04-07 22:37:20]

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

[commit-bot: I haz the power, 2017-04-07 22:38:09]

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-04-07 23:17:14]

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

[commit-bot: I haz the power, 2017-04-07 23:17:15]

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

[shivanisha, 2017-04-10 18:43:18]

On 2017/03/27 at 15:09:29, juliatuttle wrote:
> PTAL, shivanisha.

lgtm

[Julia Tuttle, 2017-04-12 16:37:32]

juliatuttle@chromium.org changed reviewers:
+ csharrison@chromium.org

[Julia Tuttle, 2017-04-12 16:37:32]

PTAL csharrison for committer signoff.

[Charlie Harrison, 2017-04-12 16:41:40]

lgtm

[Julia Tuttle, 2017-04-12 20:07:23]

The CQ bit was checked by juliatuttle@chromium.org

[Julia Tuttle, 2017-04-12 20:07:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
shivanisha@chromium.org
Link to the patchset: https://codereview.chromium.org/2769703006/#ps100001
(title: "Fix unittest failure.")

[Julia Tuttle, 2017-04-12 20:07:24]

The CQ bit was unchecked by juliatuttle@chromium.org

[Julia Tuttle, 2017-04-12 20:07:33]

The CQ bit was checked by juliatuttle@chromium.org

[Julia Tuttle, 2017-04-12 20:07:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
shivanisha@chromium.org
Link to the patchset: https://codereview.chromium.org/2769703006/#ps100001
(title: "Fix unittest failure.")

[commit-bot: I haz the power, 2017-04-12 20:08:43]

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-04-12 21:27:46]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1492027653270990,
"parent_rev": "03a5e89da19258391f9b0f06bb0d14fae46ce401", "commit_rev":
"2b8dd2d1d8c2c17e5715853ad74cf2891b3c87c7"}

[commit-bot: I haz the power, 2017-04-12 21:28:42]

Description was changed from

==========
Reporting: Plumb into UrlRequest{HttpJob,Context}.

Reporting is a spec for delivering out-of-band reports from various
other parts of the browser. See http://wicg.github.io/reporting/ for
the spec, or https://goo.gl/pygX5I for details of the planned
implementation in Chromium.

This CL adds a member in URLRequestContext{,Storage} for the
ReportingService and calls ProcessHeader from URLRequestHttpJob when
a Report-To header is received under the right circumstances.

BUG=704259
==========

to

==========
Reporting: Plumb into UrlRequest{HttpJob,Context}.

Reporting is a spec for delivering out-of-band reports from various
other parts of the browser. See http://wicg.github.io/reporting/ for
the spec, or https://goo.gl/pygX5I for details of the planned
implementation in Chromium.

This CL adds a member in URLRequestContext{,Storage} for the
ReportingService and calls ProcessHeader from URLRequestHttpJob when
a Report-To header is received under the right circumstances.

BUG=704259

Review-Url: https://codereview.chromium.org/2769703006
Cr-Commit-Position: refs/heads/master@{#464155}
Committed:
https://chromium.googlesource.com/chromium/src/+/2b8dd2d1d8c2c17e5715853ad74c...
==========

[commit-bot: I haz the power, 2017-04-12 21:28:43]

Committed patchset #6 (id:100001) as
https://chromium.googlesource.com/chromium/src/+/2b8dd2d1d8c2c17e5715853ad74c...