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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java

Issue 2806153005: Notify the remote suggestions scheduler when pulling up the Chrome Home sheet. (Closed)
Patch Set: add bug Created 3 years, 8 months 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: chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java
index cfc78b9e24fa091e302e5b6fb709fb63b09d9b02..075b65d346bd9a94714624867029e04fd742332d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/snippets/SuggestionsSource.java
@@ -115,4 +115,11 @@ public interface SuggestionsSource {
* Sets the recipient for update events from the source.
*/
void setObserver(Observer observer);
+
+ /**
+ * Notifies the scheduler to adjust the plan due to a newly opened NTP.
+ */
+ // TODO(710268): This shouldn't have "NTP" in its name, as it's not specific to the NTP anymore.
+ // Maybe split this off into a new interface for the scheduler?
+ void onNtpInitialized();
}

Powered by Google App Engine
This is Rietveld 408576698