Reporting: Add ReportingDelegate to check site permissions

Reporting: Add ReportingDelegate to check site permissions

This adds a Delegate to Reporting that can eventually be plumbed to
check whether sites are allowed to store data (reports and clients).

BUG=704259, 676324
Review-Url: https://codereview.chromium.org/2889193002
Cr-Commit-Position: refs/heads/master@{#473885}
Committed: https://chromium.googlesource.com/chromium/src/+/587548914b332653d2ae18e4d10e73ffde0f9406

[Julia Tuttle, 2017-05-18 16:09:34]

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

[Julia Tuttle, 2017-05-18 16:09:35]

PTAL, shivanisha.

[Julia Tuttle, 2017-05-18 16:41:23]

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

[commit-bot: I haz the power, 2017-05-18 16:42:19]

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-18 18:13:46]

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

[commit-bot: I haz the power, 2017-05-18 18:13:46]

Dry run: This issue passed the CQ dry run.

[Julia Tuttle, 2017-05-18 19:51:57]

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

[commit-bot: I haz the power, 2017-05-18 19:52:50]

Dry run: CQ is trying da patch.

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

[Julia Tuttle, 2017-05-18 20:05:50]

Description was changed from

==========
Reporting: Add Delegate to check site permissions

This adds a Delegate to Reporting that can eventually be plumbed to
check whether sites are allowed to store data (reports and clients).

BUG=704259,676324
==========

to

==========
Reporting: Add ReportingDelegate to check site permissions

This adds a Delegate to Reporting that can eventually be plumbed to
check whether sites are allowed to store data (reports and clients).

BUG=704259,676324
==========

[commit-bot: I haz the power, 2017-05-19 01:43:51]

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

[commit-bot: I haz the power, 2017-05-19 01:43:52]

Dry run: This issue passed the CQ dry run.

[Julia Tuttle, 2017-05-22 17:56:40]

On 2017/05/18 16:09:35, Julia Tuttle wrote:
> PTAL, shivanisha.

Ping?

[shivanisha, 2017-05-22 20:31:45]

On 2017/05/22 at 17:56:40, juliatuttle wrote:
> On 2017/05/18 16:09:35, Julia Tuttle wrote:
> > PTAL, shivanisha.
> 
> Ping?

Lgtm, thanks!

[Julia Tuttle, 2017-05-23 13:01:47]

The CQ bit was checked by juliatuttle@chromium.org

[commit-bot: I haz the power, 2017-05-23 13:02:14]

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 14:17:26]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495544507279730,
"parent_rev": "1705541fd8c739910905d53ad30c40c41b59a25d", "commit_rev":
"587548914b332653d2ae18e4d10e73ffde0f9406"}

[commit-bot: I haz the power, 2017-05-23 14:17:38]

Description was changed from

==========
Reporting: Add ReportingDelegate to check site permissions

This adds a Delegate to Reporting that can eventually be plumbed to
check whether sites are allowed to store data (reports and clients).

BUG=704259,676324
==========

to

==========
Reporting: Add ReportingDelegate to check site permissions

This adds a Delegate to Reporting that can eventually be plumbed to
check whether sites are allowed to store data (reports and clients).

BUG=704259,676324

Review-Url: https://codereview.chromium.org/2889193002
Cr-Commit-Position: refs/heads/master@{#473885}
Committed:
https://chromium.googlesource.com/chromium/src/+/587548914b332653d2ae18e4d10e...
==========

[commit-bot: I haz the power, 2017-05-23 14:17:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/587548914b332653d2ae18e4d10e...