Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1161)

Unified Diff: components/ntp_snippets/remote/persistent_scheduler.h

Issue 2557363002: [NTP Snippets] Refactor background scheduling for remote suggestions (Closed)
Patch Set: Fixing the last changes :) Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/ntp_snippets/remote/persistent_scheduler.h
diff --git a/components/ntp_snippets/remote/persistent_scheduler.h b/components/ntp_snippets/remote/persistent_scheduler.h
new file mode 100644
index 0000000000000000000000000000000000000000..70a27e74ef095fdc3a6caf0a9b39c960caba571c
--- /dev/null
+++ b/components/ntp_snippets/remote/persistent_scheduler.h
@@ -0,0 +1,47 @@
+// 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_PERSISTENT_SCHEDULER_H_
+#define COMPONENTS_NTP_SNIPPETS_REMOTE_PERSISTENT_SCHEDULER_H_
+
+#include "base/macros.h"
+#include "base/time/time.h"
+
+namespace ntp_snippets {
+
+// Interface to schedule persistent periodic fetches for remote suggestions, OS-
+// dependent. These persistent fetches must get triggered according to their
+// schedule independent of whether Chrome is running at that moment.
+//
+// Once per period, the concrete implementation should call
+// RemoteSuggestionsScheduler::OnFetchDue() where the scheduler object is
+// obtained from ContentSuggestionsService.
+//
+// The implementation may also call
+// RemoteSuggestionsScheduler::RescheduleFetching() when its own current
+// schedule got corrupted for whatever reason and needs to be applied again
+// (in turn, this will result in calling Schedule() on the implementation).
+class PersistentScheduler {
+ public:
+ // Schedule periodic fetching of remote suggestions, with different periods
+ // depending on network state. Any of the periods can be zero to indicate that
+ // the corresponding task should not be scheduled. Returns whether the
+ // scheduling was successful.
+ virtual bool Schedule(base::TimeDelta period_wifi,
+ base::TimeDelta period_fallback) = 0;
+
+ // Cancel any scheduled tasks. Equivalent to Schedule(0, 0). Returns whether
+ // the scheduling was successful.
+ virtual bool Unschedule() = 0;
+
+ protected:
+ PersistentScheduler() = default;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(PersistentScheduler);
+};
+
+} // namespace ntp_snippets
+
+#endif // COMPONENTS_NTP_SNIPPETS_REMOTE_PERSISTENT_SCHEDULER_H_

Powered by Google App Engine
This is Rietveld 408576698