Chromium Code Reviews| Index: components/metrics/data_use_tracker.cc |
| diff --git a/components/metrics/data_use_tracker.cc b/components/metrics/data_use_tracker.cc |
| index 6e3f9b3eb8afd6b77022a3094d01de6216951e6b..738656c8acf45c6edef78c66dac6e84c54b587c1 100644 |
| --- a/components/metrics/data_use_tracker.cc |
| +++ b/components/metrics/data_use_tracker.cc |
| @@ -14,6 +14,9 @@ |
| namespace metrics { |
| +const int kDefaultUMAWeeklyQuotaBytes = 204800; |
|
Alexei Svitkine (slow)
2016/05/13 18:37:13
Nit: Put these in the anon namespace below. Add co
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 |
| @@ -170,23 +173,36 @@ int DataUseTracker::ComputeTotalDataUse(const std::string& pref_name) { |
| 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; |
| } |