[Sync] Log time spent configuring data types

[Sync] Log time spent configuring data types

Add UMA histograms, too.

BUG=
TEST=

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

[akalin, 2011-05-06 23:16:35]

+zea for review

[akalin, 2011-05-07 00:35:33]

On 2011/05/06 23:16:35, akalin wrote:
> +zea for review

Fixed erroneous use of histograms.

[Nicolas Zea, 2011-05-07 00:39:35]

http://codereview.chromium.org/6948010/diff/4001/chrome/browser/sync/glue/dat...
File chrome/browser/sync/glue/data_type_manager_impl.cc (right):

http://codereview.chromium.org/6948010/diff/4001/chrome/browser/sync/glue/dat...
chrome/browser/sync/glue/data_type_manager_impl.cc:390: VLOG(1) << "NotifyDone
called with result: " << result_str;
Can we get rid of the ENUM_CASE above and just have this be in the switch
statement below with hardcoded VLOGs?

[akalin, 2011-05-09 22:19:56]

Please take another look.

http://codereview.chromium.org/6948010/diff/4001/chrome/browser/sync/glue/dat...
File chrome/browser/sync/glue/data_type_manager_impl.cc (right):

http://codereview.chromium.org/6948010/diff/4001/chrome/browser/sync/glue/dat...
chrome/browser/sync/glue/data_type_manager_impl.cc:390: VLOG(1) << "NotifyDone
called with result: " << result_str;
On 2011/05/07 00:39:37, nzea wrote:
> Can we get rid of the ENUM_CASE above and just have this be in the switch
> statement below with hardcoded VLOGs?

Done.

[Nicolas Zea, 2011-05-09 22:24:00]

LGTM.