Add histograms that monitor variation seed fetching from Java.

Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
Committed: https://crrev.com/6c139a2b0943224d42b1402584ab3c21d28d5232
Cr-Commit-Position: refs/heads/master@{#426689}

[Alexei Svitkine (slow), 2016-10-19 20:52:37]

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

[commit-bot: I haz the power, 2016-10-19 20:53:30]

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

[Alexei Svitkine (slow), 2016-10-19 21:41:35]

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

[commit-bot: I haz the power, 2016-10-19 21:42:08]

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-10-19 22:38:47]

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

[commit-bot: I haz the power, 2016-10-19 22:38:48]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)

[Alexei Svitkine (slow), 2016-10-20 14:57:07]

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

[commit-bot: I haz the power, 2016-10-20 14:57:25]

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

[Alexei Svitkine (slow), 2016-10-20 16:59:29]

Patchset #1 (id:1) has been deleted

[Alexei Svitkine (slow), 2016-10-20 16:59:45]

Patchset #1 (id:20001) has been deleted

[Alexei Svitkine (slow), 2016-10-20 16:59:52]

Description was changed from

==========
[X2] Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
==========

to

==========
Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
==========

[Alexei Svitkine (slow), 2016-10-20 17:05:24]

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

[commit-bot: I haz the power, 2016-10-20 17:05:40]

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

[Alexei Svitkine (slow), 2016-10-20 17:06:43]

Patchset #1 (id:40001) has been deleted

[Alexei Svitkine (slow), 2016-10-20 17:06:53]

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

[rkaplow, 2016-10-20 17:32:55]

lgtm

https://codereview.chromium.org/2426343004/diff/60001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2426343004/diff/60001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:67680: +    The latency of a fetching an
initial variations seed during Android Chrome
remove 'a'

[Alexei Svitkine (slow), 2016-10-20 17:35:30]

https://codereview.chromium.org/2426343004/diff/60001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2426343004/diff/60001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:67680: +    The latency of a fetching an
initial variations seed during Android Chrome
On 2016/10/20 17:32:55, rkaplow wrote:
> remove 'a'

Done.

[Alexei Svitkine (slow), 2016-10-21 00:09:02]

The CQ bit was checked by asvitkine@chromium.org

[Alexei Svitkine (slow), 2016-10-21 00:09:02]

The patchset sent to the CQ was uploaded after l-g-t-m from rkaplow@chromium.org
Link to the patchset: https://codereview.chromium.org/2426343004/#ps80001
(title: "Remove a")

[commit-bot: I haz the power, 2016-10-21 00:09:42]

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-10-21 02:16:35]

Description was changed from

==========
Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
==========

to

==========
Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
==========

[commit-bot: I haz the power, 2016-10-21 02:16:36]

Committed patchset #2 (id:80001)

[Dmitry Skiba, 2016-10-21 03:27:13]

On 2016/10/21 02:16:36, commit-bot: I haz the power wrote:
> Committed patchset #2 (id:80001)

Manually (because of the size of histograms.xm) reverted in
https://codereview.chromium.org/2437283003

[commit-bot: I haz the power, 2016-10-21 13:26:06]

Description was changed from

==========
Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199
==========

to

==========
Add histograms that monitor variation seed fetching from Java.

Depends on https://codereview.chromium.org/2435813002/ which
is moving the implementation of CachedMetrics to base from chrome/.

BUG=632199

Committed: https://crrev.com/6c139a2b0943224d42b1402584ab3c21d28d5232
Cr-Commit-Position: refs/heads/master@{#426689}
==========

[commit-bot: I haz the power, 2016-10-21 13:26:07]

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

[Alexei Svitkine (slow), 2016-10-21 20:17:43]

Relanding via: https://codereview.chromium.org/2442823002/