GitLab

This reverts commit de03f4c9.

Reason for revert: CL introduced a ReportingPermissionsChecker class which appears to be intended to safely "cancel" UI-thread work posted from the IO thread, when the class instance is destroyed. The class, however, is always destroyed on the IO thread, so this use of WeakPtrs does not have the intended effect.

Original change's description:
> Reporting: Use BACKGROUND_SYNC permission to control uploads
>
> We now use the BACKGROUND_SYNC permission to decide whether or not to
> upload reports for a particular origin.  Note that as currently written,
> we don't use this permission to decide whether to *collect* the reports,
> only whether to *send* them.  Whether or not to collect is controlled
> by the site's cookie settings.
>
> Bug: 704259
> Change-Id: I059019ab85106c26f4d156e9f5d61e2f8b8fa757
> Reviewed-on: https://chromium-review.googlesource.com/937572
> Reviewed-by: Ryan Hamilton <rch@chromium.org>
> Commit-Queue: Douglas Creager <dcreager@google.com>
> Cr-Commit-Position: refs/heads/master@{#540257}

TBR=rch@chromium.org,juliatuttle@chromium.org,dcreager@google.com

Bug: 704259
Change-Id: I9cb1990f4273f74e39de83b99a12e6bfc00e7348
Reviewed-on: https://chromium-review.googlesource.com/1056727
Commit-Queue: Wez <wez@chromium.org>
Reviewed-by: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#558160}