Expose a method in VariationsService to trigger a seed fetch.

chrome/browser/metrics/variations/variations_request_scheduler.cc

 // Copyright (c) 2013 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 "chrome/browser/metrics/variations/variations_request_scheduler.h"
 
 namespace chrome_variations {
 
 VariationsRequestScheduler::VariationsRequestScheduler(
     const base::Closure& task) : task_(task) {
@@ skipping 18 matching lines @@
 void VariationsRequestScheduler::ScheduleFetchShortly() {
   // Reset the regular timer to avoid it triggering soon after.
   Reset();
   // The delay before attempting a fetch shortly, in minutes.
   const int kFetchShortlyDelayMinutes = 5;
   one_shot_timer_.Start(FROM_HERE,
                         base::TimeDelta::FromMinutes(kFetchShortlyDelayMinutes),
                         task_);
 }
 
+void VariationsRequestScheduler::OnAppEnterForeground() {
+  // By default (non-mobile) we can reuse ScheduleFetchShortly.

[Alexei Svitkine (slow), 2014/02/13 17:36:22]

I would actually make the implementation of this be NOTREACHED();

The idea being that we don't expect this to ever be triggered on non-mobile
platforms.

[rkaplow, 2014/02/13 18:54:47]

Yeah that seems safer actually. Is this all that is needed for using
NOTREACHED()?

+  ScheduleFetchShortly();
+}
+
 base::Closure VariationsRequestScheduler::task() const {
   return task_;
 }
 
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
 // static
 VariationsRequestScheduler* VariationsRequestScheduler::Create(
     const base::Closure& task,
     PrefService* local_state) {
   return new VariationsRequestScheduler(task);
 }
 #endif  // !defined(OS_ANDROID) && !defined(OS_IOS)
 
 }  // namespace chrome_variations