[Remote suggestions] Interface for additional triggers for the scheduler

components/ntp_snippets/remote/remote_suggestions_scheduler.h

 // 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.
 
 #ifndef COMPONENTS_NTP_SNIPPETS_REMOTE_REMOTE_SUGGESTIONS_SCHEDULER_H_
 #define COMPONENTS_NTP_SNIPPETS_REMOTE_REMOTE_SUGGESTIONS_SCHEDULER_H_
 
 #include "base/macros.h"
 
 namespace ntp_snippets {
 
 // Interface for informing the scheduler.
 class RemoteSuggestionsScheduler {
  public:
+  // External triggers to consider fetching content suggestions. Returns whether
+  // a fetch will actually be triggered.
+  virtual bool OnBrowserStartup() = 0;

[tschumann, 2016/12/21 14:27:37]

is the return value actually of interest?
the caller just provides the signal, why would it care about the action the
scheduler chose?

[jkrcal, 2016/12/21 14:35:21]

Ah, you are right. I wanted to implement some histogram on the outside based on
the return value but it is actually better to do it inside the scheduler.

+  virtual bool OnNTPOpened() = 0;
+
   // Fetch content suggestions.
   virtual void OnPersistentSchedulerWakeUp() = 0;
 
   // Force rescheduling of fetching.
   virtual void RescheduleFetching() = 0;
 };
 
 }  // namespace ntp_snippets
 
 #endif  // COMPONENTS_NTP_SNIPPETS_REMOTE_REMOTE_SUGGESTIONS_SCHEDULER_H_