Only report exit funnels for canary and dev channels.

chrome/browser/metrics/chrome_metrics_service_client.cc

Index: chrome/browser/metrics/chrome_metrics_service_client.cc
diff --git a/chrome/browser/metrics/chrome_metrics_service_client.cc b/chrome/browser/metrics/chrome_metrics_service_client.cc
index 7bfc7393d3d7e20256b43830f09acbd1811c238b..5d3094134cc2106451f365ecdd04e3c89ecf2da7 100644
--- a/chrome/browser/metrics/chrome_metrics_service_client.cc
+++ b/chrome/browser/metrics/chrome_metrics_service_client.cc
@@ -296,10 +296,19 @@ void ChromeMetricsServiceClient::Initialize() {
   metrics_service_->RegisterMetricsProvider(
       scoped_ptr<metrics::MetricsProvider>(google_update_metrics_provider_));
 
+  // Report exit funnels for canary and dev only.
+  bool report_exit_funnels = false;
+  switch (chrome::VersionInfo::GetChannel()) {
+    case chrome::VersionInfo::CHANNEL_CANARY:
+    case chrome::VersionInfo::CHANNEL_DEV:
+      report_exit_funnels = true;

[Alexei Svitkine (slow), 2015/01/23 18:45:22]

Why can't this code be part of WatcherMetricsProviderWin()?

If it's just for testing, you could have two constructors for that class, one
that takes the parameter and the other that computes it...

[Sigurður Ásgeirsson, 2015/01/23 18:48:43]

The provider is a component - I didn't think it's kosher for those to take a dep
on chrome::?

[Alexei Svitkine (slow), 2015/01/23 18:50:53]

Ah, you're absolutely right!

+      break;
+  }
+
   metrics_service_->RegisterMetricsProvider(
       scoped_ptr<metrics::MetricsProvider>(
           new browser_watcher::WatcherMetricsProviderWin(
-              chrome::kBrowserExitCodesRegistryPath)));
+              chrome::kBrowserExitCodesRegistryPath, report_exit_funnels)));
 #endif  // defined(OS_WIN)
 
 #if defined(ENABLE_PLUGINS)