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 "components/ntp_snippets/features.h"
@@ skipping 120 matching lines @@
 }
 
 void SchedulingRemoteSuggestionsProvider::OnPersistentSchedulerWakeUp() {
   provider_->RefetchInTheBackground(
       base::MakeUnique<RemoteSuggestionsProvider::FetchStatusCallback>(
           base::Bind(&SchedulingRemoteSuggestionsProvider::OnFetchCompleted,
                      base::Unretained(this))));
 }
 
 void SchedulingRemoteSuggestionsProvider::OnBrowserStartup() {
+  // TODO(fhorschig): Consider that this is called twice for split screens.

[Michael van Ouwerkerk, 2016/12/23 11:04:42]

Not just twice, any number of times if the user split, closed, split again.

[fhorschig, 2017/01/03 15:04:26]

Adjusted comment to the resuming functionality.

   // TODO(jkrcal): Implement.
 }
 
 void SchedulingRemoteSuggestionsProvider::OnNTPOpened() {
   // TODO(jkrcal): Implement.
 }
 
 void SchedulingRemoteSuggestionsProvider::SetProviderStatusCallback(
       std::unique_ptr<ProviderStatusCallback> callback) {
   provider_->SetProviderStatusCallback(std::move(callback));
@@ skipping 177 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