Make the launch params the default client behavior for UMA 3g

components/metrics/data_use_tracker.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/metrics/data_use_tracker.h"
 
 #include <string>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "components/metrics/metrics_pref_names.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "components/variations/variations_associated_data.h"
 
 namespace metrics {
 
+const int kDefaultUMAWeeklyQuotaBytes = 204800;

[Alexei Svitkine (slow), 2016/05/13 18:37:13]

Nit: Put these in the anon namespace below. Add comments.

[gayane -on leave until 09-2017, 2016/05/13 19:17:01]

Done.

+const double kDefaultUMARatio = 0.05;
+
 namespace {
 
 // This function is for forwarding metrics usage pref changes to the appropriate
 // callback on the appropriate thread.
 // TODO(gayane): Reduce the frequency of posting tasks from IO to UI thread.
 void UpdateMetricsUsagePrefs(
     const UpdateUsagePrefCallbackType& update_on_ui_callback,
     scoped_refptr<base::SequencedTaskRunner> ui_task_runner,
     const std::string& service_name,
     int message_size,
@@ skipping 136 matching lines @@
     int value = 0;
     it.value().GetAsInteger(&value);
     total_data_use += value;
   }
   return total_data_use;
 }
 
 bool DataUseTracker::GetUmaWeeklyQuota(int* uma_weekly_quota_bytes) const {
   DCHECK(thread_checker_.CalledOnValidThread());
 
+  // If user is not in the experiment then there shouldn't be any limitations.
+  std::string experiment_group =
+      base::FieldTrialList::FindFullName("UMA_EnableCellularLogUpload");
+  if (experiment_group.empty())
+    return false;
+
   std::string param_value_str = variations::GetVariationParamValue(
       "UMA_EnableCellularLogUpload", "Uma_Quota");
   if (param_value_str.empty())
-    return false;
-
-  base::StringToInt(param_value_str, uma_weekly_quota_bytes);
+    *uma_weekly_quota_bytes = kDefaultUMAWeeklyQuotaBytes;
+  else
+    base::StringToInt(param_value_str, uma_weekly_quota_bytes);
   return true;
 }
 
 bool DataUseTracker::GetUmaRatio(double* ratio) const {
   DCHECK(thread_checker_.CalledOnValidThread());
 
+  // If user is not in the experiment then there shouldn't be any limitations.
+  std::string experiment_group =
+      base::FieldTrialList::FindFullName("UMA_EnableCellularLogUpload");
+  if (experiment_group.empty())
+    return false;
+
   std::string param_value_str = variations::GetVariationParamValue(
       "UMA_EnableCellularLogUpload", "Uma_Ratio");
   if (param_value_str.empty())
-    return false;
-  base::StringToDouble(param_value_str, ratio);
+    *ratio = kDefaultUMARatio;
+  else
+    base::StringToDouble(param_value_str, ratio);
   return true;
 }
 
 base::Time DataUseTracker::GetCurrentMeasurementDate() const {
   return base::Time::Now().LocalMidnight();
 }
 
 std::string DataUseTracker::GetCurrentMeasurementDateAsString() const {
   DCHECK(thread_checker_.CalledOnValidThread());
 
   base::Time::Exploded today_exploded;
   GetCurrentMeasurementDate().LocalExplode(&today_exploded);
   return base::StringPrintf("%04d-%02d-%02d", today_exploded.year,
                             today_exploded.month, today_exploded.day_of_month);
 }
 
 }  // namespace metrics