Looks like it might have been caused by this CL which refactored the code: https://codereview.chromium.org/2207933002

Hmm, yes, the call is now wrapped in a deferred task. I can do getClass().getSimpleName() outside the task.

Right, that should fix it. Probably worth evaluating if it's useful to keep the metric given no one noticed since it was broken 4 months ago?

Haha, good point. I'll send a CL to remove it and get the owner of the metric to review it.

@yusufo - are the MobileStartup.* metrics still used/relevant or can they be safely removed?

+mariakhomenko@ for whether the MobileStartup.ToolbarInflationTime.* MobileStartup.ToolbarFirstDrawTime.* and MobileStartup.ToolbarFirstFocusTime.* metrics are still used/relevant.

Thanks. I can delete them in a CL like this if they are not. https://codereview.chromium.org/2574003002/

I've actually used those metrics recently. It's actually hard to detect that things are broken unless you are restricting your view by milestone. Since we have lots of users on M53-, the metrics look quite reasonable. I would prefer these were fixed rather than removed.

e.g. https://uma.googleplex.com/p/chrome/histograms/?endDate=20161213&dayCount=1&histograms=MobileStartup.ToolbarFirstDrawTime.ChromeTabbedActivity&fixupData=true&showMax=true&filters=channel%2Ceq%2C4%2Cplatform%2Ceq%2CA%2Cisofficial%2Ceq%2CTrue&implicitFilters=isofficial

Sounds good, I'll fix these and merge these fixes.

Just wanted to make sure they are still useful. Thanks Maria.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/8f87b825a0e3cd48bf8b5f469bf25b423f903b71

commit 8f87b825a0e3cd48bf8b5f469bf25b423f903b71
Author: wnwen <wnwen@chromium.org>
Date: Thu Dec 15 17:59:37 2016

Android: Fix wrong UMA histogram name

Caused by caller switching to anonymous functions for deferred startup.

BUG= 672201 

Review-Url: https://codereview.chromium.org/2576213005
Cr-Commit-Position: refs/heads/master@{#438871}

[modify] https://crrev.com/8f87b825a0e3cd48bf8b5f469bf25b423f903b71/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java

Merge request for #10, very minor fix to UMA.

Your change meets the bar and is auto-approved for M56 (branch: 2924)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5b0d7137ef64297684ac26dac1f0e1b7a89562d4

commit 5b0d7137ef64297684ac26dac1f0e1b7a89562d4
Author: Peter Wen <wnwen@google.com>
Date: Thu Dec 15 18:38:02 2016

Android: Fix wrong UMA histogram name

Caused by caller switching to anonymous functions for deferred startup.

BUG= 672201 

Review-Url: https://codereview.chromium.org/2576213005
Cr-Commit-Position: refs/heads/master@{#438871}
(cherry picked from commit 8f87b825a0e3cd48bf8b5f469bf25b423f903b71)

Review-Url: https://codereview.chromium.org/2576283003 .
Cr-Commit-Position: refs/branch-heads/2924@{#512}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/5b0d7137ef64297684ac26dac1f0e1b7a89562d4/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java