Add FirstRun.SearchEngineBubble UMA histogram counts.

chrome/browser/ui/cocoa/first_run_bubble_controller.mm

Index: chrome/browser/ui/cocoa/first_run_bubble_controller.mm
diff --git a/chrome/browser/ui/cocoa/first_run_bubble_controller.mm b/chrome/browser/ui/cocoa/first_run_bubble_controller.mm
index 26ac64e5982e56055b0563ec264f0f02c7a168ed..aa727a20d2b7e8766e0f95fc2984ee0c2b3c6179 100644
--- a/chrome/browser/ui/cocoa/first_run_bubble_controller.mm
+++ b/chrome/browser/ui/cocoa/first_run_bubble_controller.mm
@@ -5,7 +5,9 @@
 #import "chrome/browser/ui/cocoa/first_run_bubble_controller.h"
 
 #include "base/logging.h"
+#include "base/metrics/histogram.h"
 #include "base/utf_string_conversions.h"
+#include "chrome/browser/first_run/first_run.h"
 #include "chrome/browser/search_engines/util.h"
 #include "chrome/browser/ui/browser_list.h"
 #import "chrome/browser/ui/cocoa/l10n_util.h"
@@ -51,6 +53,10 @@
 }
 
 - (void)awakeFromNib {
+  HISTOGRAM_ENUMERATION("FirstRun.SearchEngineBubble",

[Ilya Sherman, 2012/04/04 23:28:35]

This should be an UMA_HISTOGRAM_ENUMERATION, so that the data gets uploaded to
UMA (versus just collected locally).

[msw, 2012/04/04 23:50:40]

Done.

+                        first_run::FIRST_RUN_BUBBLE_SHOWN,
+                        first_run::NUM_FIRST_RUN_BUBBLE_METRICS);
+
   DCHECK(header_);
   [header_ setStringValue:cocoa_l10n_util::ReplaceNSStringPlaceholders(
       [header_ stringValue], GetDefaultSearchEngineName(profile_), NULL)];
@@ -81,6 +87,10 @@
 }
 
 - (IBAction)onChange:(id)sender {
+  HISTOGRAM_ENUMERATION("FirstRun.SearchEngineBubble",

[Ilya Sherman, 2012/04/04 23:28:35]

nit: I would recommend having a single LogFirstRunMetric() function exposed,
perhaps in chrome/browser/first_run/first_run.h, so that there's no opportunity
for typos to creep into the histogram name.

[msw, 2012/04/04 23:50:40]

Done.

+                        first_run::FIRST_RUN_BUBBLE_CHANGE_SEARCH_INVOKED,
+                        first_run::NUM_FIRST_RUN_BUBBLE_METRICS);
+
   Browser* browser = BrowserList::GetLastActiveWithProfile(profile_);
   [self close];
   if (browser)