Chromium Code Reviews| Index: chrome/browser/metrics/perf/perf_provider_chromeos.cc |
| diff --git a/chrome/browser/metrics/perf/perf_provider_chromeos.cc b/chrome/browser/metrics/perf/perf_provider_chromeos.cc |
| index 1d782ac185c413be5d5a4392ab16a8e0be9f5387..045cae21c7d166361d29de7518876babdddbbf38 100644 |
| --- a/chrome/browser/metrics/perf/perf_provider_chromeos.cc |
| +++ b/chrome/browser/metrics/perf/perf_provider_chromeos.cc |
| @@ -190,6 +190,17 @@ std::vector<RandomSelector::WeightAndValue> GetDefaultCommandsForCpu( |
| } // namespace internal |
| +PerfProvider::CollectionParams::CollectionParams() |
| + : CollectionParams(base::TimeDelta::FromSeconds(2), |
|
dhsharp
2016/02/27 03:05:24
Please at least restore the comments for what thes
Lei Zhang
2016/03/29 19:03:45
Done.
|
| + base::TimeDelta::FromHours(3), |
| + PerfProvider::CollectionParams::TriggerParams( |
| + 10, |
| + base::TimeDelta::FromSeconds(5)), |
| + PerfProvider::CollectionParams::TriggerParams( |
| + 10, |
| + base::TimeDelta::FromSeconds(10))) { |
| +} |
| + |
| PerfProvider::CollectionParams::CollectionParams( |
| base::TimeDelta collection_duration, |
| base::TimeDelta periodic_interval, |
| @@ -207,19 +218,8 @@ PerfProvider::CollectionParams::TriggerParams::TriggerParams( |
| : sampling_factor_(sampling_factor), |
| max_collection_delay_(max_collection_delay.ToInternalValue()) {} |
| -const PerfProvider::CollectionParams PerfProvider::kDefaultParameters( |
| - /* collection_duration = */ base::TimeDelta::FromSeconds(2), |
| - /* periodic_interval = */ base::TimeDelta::FromHours(3), |
| - /* resume_from_suspend = */ PerfProvider::CollectionParams::TriggerParams( |
| - /* sampling_factor = */ 10, |
| - /* max_collection_delay = */ base::TimeDelta::FromSeconds(5)), |
| - /* restore_session = */ PerfProvider::CollectionParams::TriggerParams( |
| - /* sampling_factor = */ 10, |
| - /* max_collection_delay = */ base::TimeDelta::FromSeconds(10))); |
| - |
| PerfProvider::PerfProvider() |
| - : collection_params_(kDefaultParameters), |
| - login_observer_(this), |
| + : login_observer_(this), |
| next_profiling_interval_start_(base::TimeTicks::Now()), |
| weak_factory_(this) { |
| } |