Upload UMA data using protocol buffers.

chrome/browser/chrome_browser_main.cc

Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index c03bc0d1471f2156185ddf6ce41a0d28662c3b7f..f283c330e351234ed25cfcfb31b7ad668f32f158 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -1054,6 +1054,7 @@ void ChromeBrowserMainParts::StartMetricsRecording() {
 }
 
 bool ChromeBrowserMainParts::IsMetricsReportingEnabled() {
+  return true;

[Ilya Sherman, 2012/02/17 06:07:44]

TODO(isherman): This is handy for testing for now... should definitely be
removed prior to committing.

[jar (doing other things), 2012/02/23 01:59:18]

I had proposed putting in this exact code in a:
#if !defined(NDEBUG)

This would facilitate testing and developing... and allow us to drop in
interesting DCHECKs to prevent regressions.

WDYT?


On 2012/02/17 06:07:44, Ilya Sherman wrote:

[Ilya Sherman, 2012/02/24 02:10:06]

Sounds fine to me, but probably best to do in a separate CL.  I think MAD is
planning to make this change sometime this week, when he gets around to it...

[jar (doing other things), 2012/02/27 20:35:34]

Since it would be a bad thing to check this in by accident, I'd suggest that you
at least (tentatively) enclose it in an if !defined(NDEBUG).  If you checked it
in by accident, it would be no big deal that way.

I think it makes it much easier to test this way (as you noted).


On 2012/02/24 02:10:06, Ilya Sherman wrote:

[Ilya Sherman, 2012/02/28 00:23:10]

D'oh.  Re-removed from this CL, and added the "#if !defined(NDEBUG)" wrapper to
my local diff file that I patch in for testing.

   // If the user permits metrics reporting with the checkbox in the
   // prefs, we turn on recording.  We disable metrics completely for
   // non-official builds.