Chromium Code Reviews| Index: chrome/browser/metrics/variations/variations_request_scheduler_mobile.cc |
| diff --git a/chrome/browser/metrics/variations/variations_request_scheduler_mobile.cc b/chrome/browser/metrics/variations/variations_request_scheduler_mobile.cc |
| index 77148617bfcb9747711027767e5da2ae427bdca2..b97cc11a81320151ae0402f047c21f8fc777a6ff 100644 |
| --- a/chrome/browser/metrics/variations/variations_request_scheduler_mobile.cc |
| +++ b/chrome/browser/metrics/variations/variations_request_scheduler_mobile.cc |
| @@ -11,6 +11,9 @@ namespace chrome_variations { |
| namespace { |
| +// Time before attempting a seed fetch after a ScheduleFetch(), in seconds. |
| +const int kScheduleFetchDelaySeconds = 5; |
| + |
| // Time between seed fetches, in hours. |
| const int kSeedFetchPeriodHours = 5; |
| @@ -41,6 +44,17 @@ void VariationsRequestSchedulerMobile::Start() { |
| void VariationsRequestSchedulerMobile::Reset() { |
| } |
| +void VariationsRequestSchedulerMobile::ScheduleFetch() { |
|
Alexei Svitkine (slow)
2014/02/12 21:17:44
What's the reason for this being separate from Sch
rkaplow
2014/02/12 22:51:52
ScheduleFetchShortly()/ForceFetch() is doing somet
|
| + // Since Start() launches a one-off execution, we can reuse it here. Also |
| + // note that since Start() verifies that the seed needs to be refreshed, we |
| + // do not verify here. |
| + schedule_fetch_timer_.Start( |
| + FROM_HERE, |
| + base::TimeDelta::FromSeconds(kScheduleFetchDelaySeconds), |
| + this, |
| + &VariationsRequestSchedulerMobile::Start); |
| +} |
| + |
| // static |
| VariationsRequestScheduler* VariationsRequestScheduler::Create( |
| const base::Closure& task, |