Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2167)

Unified Diff: chrome/browser/metrics/metrics_service.cc

Issue 7056031: Collect stats to investigate the viability of UDP (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/metrics/metrics_service.cc
===================================================================
--- chrome/browser/metrics/metrics_service.cc (revision 87905)
+++ 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();
@@ -1365,6 +1369,10 @@
bool server_is_healthy = upload_succeeded || response_code == 400;
scheduler_->UploadFinished(server_is_healthy, unsent_logs());
+
+ // Collect network stats if UMA upload succeeded.
+ if (server_is_healthy)
+ chrome_browser_net::CollectNetworkStats(network_stats_server_);
}
void MetricsService::LogBadResponseCode() {

Powered by Google App Engine
This is Rietveld 408576698