Move Java CachedMetrics functionality to base/.

Move Java CachedMetrics functionality to base/.

This is general-purpose functionality that allows
recording UMA metrics before the C++ native library
has been loaded and is useful to have outside of the
Chrome layer. In particular, I will be using it from
components/variations to record information about the
first run variations seed fetch.

No functional changes in this CL.

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

[Alexei Svitkine (slow), 2016-10-19 17:48:59]

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

[commit-bot: I haz the power, 2016-10-19 17:49:32]

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 18:28:37]

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

[commit-bot: I haz the power, 2016-10-19 18:28:38]

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

[Alexei Svitkine (slow), 2016-10-19 18:40:06]

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

[Alexei Svitkine (slow), 2016-10-19 18:40:24]

Patchset #1 (id:1) has been deleted

[commit-bot: I haz the power, 2016-10-19 18:40: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-19 19:21:04]

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

[Alexei Svitkine (slow), 2016-10-19 19:23:19]

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

[commit-bot: I haz the power, 2016-10-19 19:23:36]

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 19:24:14]

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

[Alexei Svitkine (slow), 2016-10-19 19:24:15]

bauerb: overall review
nyquist: base/android OWNERS review

Thanks!

[Alexei Svitkine (slow), 2016-10-19 20:16:11]

Seems all of base/android OWNERS are OOO this week. So I might have to switch
out nyquist@ for someone in base/OWNERS.

Anyway, Bernhard can you still do an overall review since you've worked on the
code I'm moving and let me know if moving this to base/ sounds good to you.

[commit-bot: I haz the power, 2016-10-19 22:33:29]

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

[commit-bot: I haz the power, 2016-10-19 22:33:30]

Dry run: This issue passed the CQ dry run.

[Bernhard Bauer, 2016-10-20 12:45:59]

Nice! LGTM

[Alexei Svitkine (slow), 2016-10-20 15:15:55]

asvitkine@chromium.org changed reviewers:
+ thestig@chromium.org
- nyquist@chromium.org

[Alexei Svitkine (slow), 2016-10-20 15:15:56]

+thestig for base/ OWNERS since everyone in base/android/OWNERS is OOO this
week.

This change has already been reviewed by Bernhard who's an owner of
chrome/android and is just moving code to base/ that has to do with recording
histograms (something already implemented in base/), so I think is non
controversial.

[Lei Zhang, 2016-10-20 21:10:54]

rs lgtm

[Alexei Svitkine (slow), 2016-10-20 21:11:39]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2016-10-20 21:12:09]

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-20 23:48:50]

Committed patchset #2 (id:40001)

[Dmitry Skiba, 2016-10-21 02:16:05]

A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/2443483003/ by dskiba@google.com.

The reason for reverting is: broke Clank: ExternalAuthUtilsInternal depends on
LaunchMetrics.TimesHistogramSample.

[DmitrySkiba, 2016-10-21 02:28:33]

A revert of this CL (patchset #2 id:40001) has been created in
https://codereview.chromium.org/2442503005/ by dskiba@chromium.org.

The reason for reverting is: Broke Clank: ExternalAuthUtilsInternal depends on
LaunchMetrics.TimesHistogramSample..

[commit-bot: I haz the power, 2016-10-21 13:24:42]

Description was changed from

==========
Move Java CachedMetrics functionality to base/.

This is general-purpose functionality that allows
recording UMA metrics before the C++ native library
has been loaded and is useful to have outside of the
Chrome layer. In particular, I will be using it from
components/variations to record information about the
first run variations seed fetch.

No functional changes in this CL.

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

to

==========
Move Java CachedMetrics functionality to base/.

This is general-purpose functionality that allows
recording UMA metrics before the C++ native library
has been loaded and is useful to have outside of the
Chrome layer. In particular, I will be using it from
components/variations to record information about the
first run variations seed fetch.

No functional changes in this CL.

BUG=632199

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

[commit-bot: I haz the power, 2016-10-21 13:24:43]

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

[Alexei Svitkine (slow), 2016-10-21 16:16:21]

Patchset #3 (id:60001) has been deleted

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

Relanded as https://codereview.chromium.org/2439113002/