Android FRE: Use CachedMetrics API for histograms.

Android FRE: Use CachedMetrics API for histograms.

I'm working on making it possible to show the first
FRE screen before native is initialized.

RecordHistograms API requires native to be initialized.
This switches to using CachedMetrics API which supports
logging before then.

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

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

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

[commit-bot: I haz the power, 2016-10-31 21:20:38]

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-31 21:51:08]

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

[commit-bot: I haz the power, 2016-10-31 21:51:09]

Dry run: This issue passed the CQ dry run.

[Alexei Svitkine (slow), 2016-11-01 15:10:07]

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

[gogerald1, 2016-11-02 13:58:15]

lgtm with a comment

https://codereview.chromium.org/2466723002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunActivity.java
(right):

https://codereview.chromium.org/2466723002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunActivity.java:81:
private static final String FRE_ENTRY_VIEW_INTENT = ".ViewIntent";
remove this line?

[Alexei Svitkine (slow), 2016-11-02 15:36:36]

https://codereview.chromium.org/2466723002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunActivity.java
(right):

https://codereview.chromium.org/2466723002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunActivity.java:81:
private static final String FRE_ENTRY_VIEW_INTENT = ".ViewIntent";
On 2016/11/02 13:58:15, gogerald1 wrote:
> remove this line?

Done.

[Alexei Svitkine (slow), 2016-11-02 15:43:52]

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

[Alexei Svitkine (slow), 2016-11-02 15:43:52]

+tedchoc for owners

[Ted C, 2016-11-02 16:46:00]

lgtm

[Alexei Svitkine (slow), 2016-11-02 16:47:29]

The CQ bit was checked by asvitkine@chromium.org

[Alexei Svitkine (slow), 2016-11-02 16:47:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
gogerald@chromium.org
Link to the patchset: https://codereview.chromium.org/2466723002/#ps20001
(title: "Address comment.")

[commit-bot: I haz the power, 2016-11-02 16:48:05]

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

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-02 17:56:25]

Description was changed from

==========
Android FRE: Use CachedMetrics API for histograms.

I'm working on making it possible to show the first
FRE screen before native is initialized.

RecordHistograms API requires native to be initialized.
This switches to using CachedMetrics API which supports
logging before then.

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

to

==========
Android FRE: Use CachedMetrics API for histograms.

I'm working on making it possible to show the first
FRE screen before native is initialized.

RecordHistograms API requires native to be initialized.
This switches to using CachedMetrics API which supports
logging before then.

BUG=632199

Committed: https://crrev.com/2770c135dab4859008af3b73c0cc7ed2bea98acc
Cr-Commit-Position: refs/heads/master@{#429317}
==========

[commit-bot: I haz the power, 2016-11-02 17:56:26]

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