base::Bind() conversion for MetricsService.

base::Bind() conversion for MetricsService.

BUG=none
TEST=trybots


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111047

[jar (doing other things), 2011-11-21 17:11:58]

isherman: PTAL

I could have sworn I did a review for a CL on this topic in the past week or
two.  I think it was by isherman, so I'm copying him.  If he has no objections
(and no CL about to land!), then LGTM.

[Ilya Sherman, 2011-11-21 21:09:46]

LGTM.  I held off on updating these last few call sites in my CL because I
didn't realize that Callbacks had is_null() and Reset() methods defined, and
hence thought these changes would need to be a bit more involved.

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
File chrome/browser/metrics/histogram_synchronizer.h (right):

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
chrome/browser/metrics/histogram_synchronizer.h:82: int wait_time);
nit: As long as you're updating this declaration, please also update it to have
one parameter per line.

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
chrome/browser/metrics/histogram_synchronizer.h:123: int unresponsive_renderers,
const base::TimeTicks& started);
nit: Here too.

[dcheng, 2011-11-21 22:06:01]

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
File chrome/browser/metrics/histogram_synchronizer.h (right):

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
chrome/browser/metrics/histogram_synchronizer.h:82: int wait_time);
On 2011/11/21 21:09:46, Ilya Sherman wrote:
> nit: As long as you're updating this declaration, please also update it to
have
> one parameter per line.

Done.

http://codereview.chromium.org/8603013/diff/2002/chrome/browser/metrics/histo...
chrome/browser/metrics/histogram_synchronizer.h:123: int unresponsive_renderers,
const base::TimeTicks& started);
On 2011/11/21 21:09:46, Ilya Sherman wrote:
> nit: Here too.

Done.

[commit-bot: I haz the power, 2011-11-21 22:06:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dcheng@chromium.org/8603013/9001

[commit-bot: I haz the power, 2011-11-22 00:08:41]

Change committed as 111047