Domain Reliability: Add more upload-related histograms.

Domain Reliability: Add more upload-related histograms.

Add some more histograms characterizing the frequency and distribution of
uploads, so we can see how well the backoff behavior works in production.

BUG=660989
Committed: https://crrev.com/dd99ca65f3c77b2de0d8d5a43c77d706152422c8
Cr-Commit-Position: refs/heads/master@{#429053}

[Julia Tuttle, 2016-10-31 23:19:52]

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

[Julia Tuttle, 2016-10-31 23:20:16]

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

[Julia Tuttle, 2016-10-31 23:20:17]

PTAL, asvitkine.

[commit-bot: I haz the power, 2016-10-31 23:20:37]

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

[Julia Tuttle, 2016-10-31 23:32:06]

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
File components/domain_reliability/context_manager.cc (left):

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
components/domain_reliability/context_manager.cc:56: DVLOG(1) <<
"Adding/replacing context for existing origin " << origin.spec()
Note to self: Put this back.

[commit-bot: I haz the power, 2016-11-01 00:42:44]

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

[commit-bot: I haz the power, 2016-11-01 00:42:45]

Dry run: This issue passed the CQ dry run.

[Alexei Svitkine (slow), 2016-11-01 14:49:46]

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
File components/domain_reliability/context_manager.cc (right):

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
components/domain_reliability/context_manager.cc:67: true);
Please refactor the code to avoid having two macro invocations for the same
histogram. Each macro expands to a bunch of code which bloats the binary.

[Julia Tuttle, 2016-11-01 15:52:37]

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

[Julia Tuttle, 2016-11-01 15:52:47]

PTAL, asvitkine.

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
File components/domain_reliability/context_manager.cc (left):

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
components/domain_reliability/context_manager.cc:56: DVLOG(1) <<
"Adding/replacing context for existing origin " << origin.spec()
On 2016/10/31 23:32:05, Julia Tuttle wrote:
> Note to self: Put this back.

Done.

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
File components/domain_reliability/context_manager.cc (right):

https://codereview.chromium.org/2466093002/diff/1/components/domain_reliabili...
components/domain_reliability/context_manager.cc:67: true);
On 2016/11/01 14:49:46, Alexei Svitkine (slow) wrote:
> Please refactor the code to avoid having two macro invocations for the same
> histogram. Each macro expands to a bunch of code which bloats the binary.

Done.

[commit-bot: I haz the power, 2016-11-01 15:52:59]

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, 2016-11-01 16:50:26]

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

[commit-bot: I haz the power, 2016-11-01 16:50:26]

Dry run: This issue passed the CQ dry run.

[Alexei Svitkine (slow), 2016-11-01 17:02:53]

lgtm

[Julia Tuttle, 2016-11-01 18:06:04]

The CQ bit was checked by juliatuttle@chromium.org

[commit-bot: I haz the power, 2016-11-01 18:06:23]

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-11-01 18:12:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-01 18:24:59]

Description was changed from

==========
Domain Reliability: Add more upload-related histograms.

Add some more histograms characterizing the frequency and distribution of
uploads, so we can see how well the backoff behavior works in production.

BUG=660989
==========

to

==========
Domain Reliability: Add more upload-related histograms.

Add some more histograms characterizing the frequency and distribution of
uploads, so we can see how well the backoff behavior works in production.

BUG=660989

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

[commit-bot: I haz the power, 2016-11-01 18:25:00]

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