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

Side by Side Diff: chrome/browser/net/network_stats.cc

Issue 102763004: Enable the network connectivity tests. Changes in CL 10299300 would fix (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/net/network_stats.h" 5 #include "chrome/browser/net/network_stats.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/message_loop/message_loop.h" 9 #include "base/message_loop/message_loop.h"
10 #include "base/metrics/field_trial.h" 10 #include "base/metrics/field_trial.h"
(...skipping 867 matching lines...) Expand 10 before | Expand all | Expand 10 after
878 // Enable the connectivity testing for 50% of the users in canary channel. 878 // Enable the connectivity testing for 50% of the users in canary channel.
879 probability_per_group = kDivisor / 2; 879 probability_per_group = kDivisor / 2;
880 } else if (channel == chrome::VersionInfo::CHANNEL_DEV) { 880 } else if (channel == chrome::VersionInfo::CHANNEL_DEV) {
881 // Enable the connectivity testing for 10% of the users in dev channel. 881 // Enable the connectivity testing for 10% of the users in dev channel.
882 probability_per_group = kDivisor / 10; 882 probability_per_group = kDivisor / 10;
883 } else if (channel == chrome::VersionInfo::CHANNEL_BETA) { 883 } else if (channel == chrome::VersionInfo::CHANNEL_BETA) {
884 // Enable the connectivity testing for 1% of the users in beta channel. 884 // Enable the connectivity testing for 1% of the users in beta channel.
885 probability_per_group = kDivisor / 100; 885 probability_per_group = kDivisor / 100;
886 } 886 }
887 887
888 // TODO(rtenneti): Enable the experiment after fixing
889 // issue http://crbug.com/273917.
890 probability_per_group = 0;
891
892 // After July 31, 2014 builds, it will always be in default group 888 // After July 31, 2014 builds, it will always be in default group
893 // (disable_network_stats). 889 // (disable_network_stats).
894 trial = base::FieldTrialList::FactoryGetFieldTrial( 890 trial = base::FieldTrialList::FactoryGetFieldTrial(
895 "NetworkConnectivity", kDivisor, "disable_network_stats", 891 "NetworkConnectivity", kDivisor, "disable_network_stats",
896 2014, 7, 31, base::FieldTrial::SESSION_RANDOMIZED, NULL); 892 2014, 7, 31, base::FieldTrial::SESSION_RANDOMIZED, NULL);
897 893
898 // Add option to collect_stats for NetworkConnectivity. 894 // Add option to collect_stats for NetworkConnectivity.
899 int collect_stats_group = 895 int collect_stats_group =
900 trial->AppendGroup("collect_stats", probability_per_group); 896 trial->AppendGroup("collect_stats", probability_per_group);
901 if (trial->group() == collect_stats_group) 897 if (trial->group() == collect_stats_group)
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
949 udp_stats_client->Start(host_resolver, 945 udp_stats_client->Start(host_resolver,
950 server_address, 946 server_address,
951 histogram_port, 947 histogram_port,
952 has_proxy_server, 948 has_proxy_server,
953 kProbePacketBytes[probe_choice], 949 kProbePacketBytes[probe_choice],
954 bytes_for_packet_size_test, 950 bytes_for_packet_size_test,
955 net::CompletionCallback()); 951 net::CompletionCallback());
956 } 952 }
957 953
958 } // namespace chrome_browser_net 954 } // namespace chrome_browser_net
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698