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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java

Issue 1912063003: [NTP Snippets] Stop refreshing the snippets on tab switch (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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/cards/NewTabPageAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
index de7aeeefe90c611aa3b2fd9dc82756b9b79616d2..971f58506ebf153d1bc9986aeb244c7711b2207e 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageAdapter.java
@@ -17,8 +17,6 @@
import org.chromium.chrome.browser.ntp.snippets.SnippetArticleViewHolder;
import org.chromium.chrome.browser.ntp.snippets.SnippetHeaderListItem;
import org.chromium.chrome.browser.ntp.snippets.SnippetsBridge.SnippetsObserver;
-import org.chromium.chrome.browser.tab.EmptyTabObserver;
-import org.chromium.chrome.browser.tab.Tab;
import java.util.ArrayList;
import java.util.List;
@@ -81,14 +79,6 @@ public NewTabPageAdapter(NewTabPageManager manager, NewTabPageLayout newTabPageL
mNewTabPageListItems.add(mAboveTheFoldListItem);
mNewTabPageManager.setSnippetsObserver(this);
-
- // Fetch new snippets when switching back to the NTP, from another tab or another app.
- mNewTabPageManager.addTabObserver(new EmptyTabObserver() {
- @Override
- public void onShown(Tab tab) {
- mNewTabPageManager.setSnippetsObserver(NewTabPageAdapter.this);
- }
- });
}
/** Returns callbacks to configure the interactions with the RecyclerView's items. */

Powered by Google App Engine
This is Rietveld 408576698