Trigger the Snippets Scheduler when Resuming Activities

components/ntp_snippets/remote/scheduling_remote_suggestions_provider.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/ntp_snippets/remote/scheduling_remote_suggestions_provider.h"
 
 #include <string>
 #include <utility>
 
 #include "base/memory/ptr_util.h"
@@ skipping 120 matching lines @@
   StartScheduling();
 }
 
 void SchedulingRemoteSuggestionsProvider::OnPersistentSchedulerWakeUp() {
   provider_->RefetchInTheBackground(
       base::MakeUnique<RemoteSuggestionsProvider::FetchStatusCallback>(
           base::Bind(&SchedulingRemoteSuggestionsProvider::OnFetchCompleted,
                      base::Unretained(this))));
 }
 
-void SchedulingRemoteSuggestionsProvider::OnBrowserStartup() {
+void SchedulingRemoteSuggestionsProvider::OnBrowserForegrounded() {
+  // TODO(fhorschig|jkrcal): Consider that this is called whenever we open or
+  // return to an Activity. Therefore, keep work light for fast start up calls.
+  // TODO(jkrcal): Implement.
+}
+
+void SchedulingRemoteSuggestionsProvider::OnBrowserColdStart() {
+  // TODO(fhorschig|jkrcal): Consider that work here must be kept light for fast
+  // cold start ups.
   // TODO(jkrcal): Implement.
 }
 
 void SchedulingRemoteSuggestionsProvider::OnNTPOpened() {
+  // TODO(fhorschig|jkrcal): Consider that this may be called on cold starts,
+  // so keep the work light for this case.
   // TODO(jkrcal): Implement.
 }
 
 void SchedulingRemoteSuggestionsProvider::SetProviderStatusCallback(
       std::unique_ptr<ProviderStatusCallback> callback) {
   provider_->SetProviderStatusCallback(std::move(callback));
 }
 
 void SchedulingRemoteSuggestionsProvider::RefetchInTheBackground(
     std::unique_ptr<FetchStatusCallback> callback) {
@@ skipping 173 matching lines @@
   }
 
   // Reschedule after a successful fetch. This resets all currently scheduled
   // fetches, to make sure the fallback interval triggers only if no wifi fetch
   // succeeded, and also that we don't do a background fetch immediately after
   // a user-initiated one.
   ApplyFetchingSchedule(GetLastFetchingSchedule());
 }
 
 }  // namespace ntp_snippets