Reporting: Document browsing data remover + endpoint backoff

Reporting: Document browsing data remover + endpoint backoff

The ReportingBrowsingDataRemover currently doesn't remove entries from
the endpoint backoff map in the ReportingEndpointManager, because it's
never persisted to disk. This CL adds comments in a couple places to
note this, so someone implementing persistence for that knows to
implement browsing data removal as well.

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

[Julia Tuttle, 2017-05-04 17:12:16]

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

[Julia Tuttle, 2017-05-04 17:12:17]

PTAL, shivanisha.

[Julia Tuttle, 2017-05-08 16:39:53]

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

[commit-bot: I haz the power, 2017-05-08 16:40:45]

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-08 17:02:00]

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

[commit-bot: I haz the power, 2017-05-08 17:02:01]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[shivanisha, 2017-05-09 14:00:49]

On 2017/05/04 at 17:12:17, juliatuttle wrote:
> PTAL, shivanisha.

lgtm

[Julia Tuttle, 2017-05-09 14:32:11]

The CQ bit was checked by juliatuttle@chromium.org

[commit-bot: I haz the power, 2017-05-09 14:32:38]

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-09 15:05:45]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494340331218320,
"parent_rev": "f6642562d6be03d4a3e03f6821b5b52ad6493da1", "commit_rev":
"cf0772c92c9909b3ad0f107541a533c567bfda75"}

[commit-bot: I haz the power, 2017-05-09 15:05:58]

Description was changed from

==========
Reporting: Document browsing data remover + endpoint backoff

The ReportingBrowsingDataRemover currently doesn't remove entries from
the endpoint backoff map in the ReportingEndpointManager, because it's
never persisted to disk. This CL adds comments in a couple places to
note this, so someone implementing persistence for that knows to
implement browsing data removal as well.

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

to

==========
Reporting: Document browsing data remover + endpoint backoff

The ReportingBrowsingDataRemover currently doesn't remove entries from
the endpoint backoff map in the ReportingEndpointManager, because it's
never persisted to disk. This CL adds comments in a couple places to
note this, so someone implementing persistence for that knows to
implement browsing data removal as well.

BUG=704259,676324

Review-Url: https://codereview.chromium.org/2862003003
Cr-Commit-Position: refs/heads/master@{#470336}
Committed:
https://chromium.googlesource.com/chromium/src/+/cf0772c92c9909b3ad0f107541a5...
==========

[commit-bot: I haz the power, 2017-05-09 15:05:59]

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