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 <random>
 #include <string>
 #include <utility>
 
@@ skipping 162 matching lines @@
 }
 
 void SchedulingRemoteSuggestionsProvider::OnPersistentSchedulerWakeUp() {
   if (BackgroundFetchesDisabled()) {
     return;
   }
 
   RefetchInTheBackground(/*callback=*/nullptr);
 }
 
-void SchedulingRemoteSuggestionsProvider::OnBrowserStartup() {
+void SchedulingRemoteSuggestionsProvider::OnBrowserForegrounded() {
+  // TODO(fhorschig|jkrcal): Consider that this is called whenever we open or

[Bernhard Bauer, 2017/01/05 16:32:54]

This syntax doesn't lend itself very well to grepping. Given that the purpose of
a name in a TODO is to state the point of contact and not the person assigned to
do it, can you just put a single name in there?

[fhorschig, 2017/01/05 17:11:13]

Changed. Thank you for explaining, I didn't know this purpose.

+  // 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() {
   if (BackgroundFetchesDisabled() || !ShouldRefetchInTheBackgroundNow()) {
     return;
   }
 
   RefetchInTheBackground(/*callback=*/nullptr);
 }
 
 void SchedulingRemoteSuggestionsProvider::SetProviderStatusCallback(
     std::unique_ptr<ProviderStatusCallback> callback) {
@@ skipping 212 matching lines @@
   // successful fetch. After a failed fetch, we want to keep the previous
   // persistent schedule intact so that we eventually get a persistent
   // fallback fetch (if the wifi persistent fetches keep failing).
   if (fetch_status.code != StatusCode::SUCCESS) {
     return;
   }
   ApplyPersistentFetchingSchedule();
 }
 
 }  // namespace ntp_snippets