Collect stats to investigate the viability of UDP

chrome/browser/metrics/metrics_service.cc

Index: chrome/browser/metrics/metrics_service.cc
===================================================================
--- chrome/browser/metrics/metrics_service.cc	(revision 86778)
+++ chrome/browser/metrics/metrics_service.cc	(working copy)
@@ -174,6 +174,7 @@
 #include "chrome/browser/metrics/histogram_synchronizer.h"
 #include "chrome/browser/metrics/metrics_log.h"
 #include "chrome/browser/metrics/metrics_reporting_scheduler.h"
+#include "chrome/browser/net/network_stats.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
 #include "chrome/browser/profiles/profile.h"
@@ -694,9 +695,12 @@
 void MetricsService::InitializeMetricsState() {
 #if defined(OS_POSIX)
   server_url_ = L"https://clients4.google.com/firefox/metrics/collect";
+  // TODO(rtenneti): Return the network stats server name.
+  network_stats_server_ = "";
 #else
   BrowserDistribution* dist = BrowserDistribution::GetDistribution();
   server_url_ = dist->GetStatsServerURL();
+  network_stats_server_ = dist->GetNetworkStatsServer();
 #endif
 
   PrefService* pref = g_browser_process->local_state();
@@ -1027,10 +1031,11 @@
   waiting_for_asynchronus_reporting_step_ = true;
   current_fetch_->Start();
 
+  chrome_browser_net::CollectNetworkStats(network_stats_server_);

[Mike Belshe, 2011/05/27 21:27:00]

How often do you want to run this test?  I'm not sure how often the
MetricsService::OnHistogramSynchronizationDone() is called?  Is this once per
browser lifetime?  Or is it repeated?

My thoughts for when to schedule a test:
   - field trial selects 0.5% of users
   - randomly pick a time between 10 and 20 minutes after browser start
   - run test once

But I made that up - other suggestions welcome.

[ramant (doing other things), 2011/05/31 21:25:36]

Made changes per jar and your talks. CollectStats after successful UMA upload.
We upload UMA stats 1 minute after startup and every 30 mins after that.

Done.

+
   HandleIdleSinceLastTransmission(true);
 }
 
-
 void MetricsService::MakePendingLog() {
   if (pending_log())
     return;