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

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

Issue 2622793003: 📰 Implement offline badge refresh via partial bind (Closed)
Patch Set: address comments Created 3 years, 11 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 3299b3475d17f7421b75be266886590da408112e..eadf900e456c1b6c2de7282947093781bfa687b4 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
@@ -20,6 +20,9 @@
import org.chromium.chrome.browser.ntp.snippets.SnippetArticleViewHolder;
import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
+import java.util.Collections;
+import java.util.List;
+
/**
* A class that handles merging above the fold elements and below the fold cards into an adapter
* that will be used to back the NTP RecyclerView. The first element in the adapter should always be
@@ -131,8 +134,13 @@ public NewTabPageViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
}
@Override
+ public void onBindViewHolder(NewTabPageViewHolder holder, int position, List<Object> payloads) {
+ mRoot.onBindViewHolder(holder, position, payloads);
+ }
+
+ @Override
public void onBindViewHolder(NewTabPageViewHolder holder, final int position) {
- mRoot.onBindViewHolder(holder, position);
+ mRoot.onBindViewHolder(holder, position, Collections.emptyList());
}
@Override
@@ -174,9 +182,10 @@ private void updateAllDismissedVisibility() {
}
@Override
- public void onItemRangeChanged(TreeNode child, int itemPosition, int itemCount) {
+ public void onItemRangeChanged(
+ TreeNode child, int itemPosition, int itemCount, Object payload) {
assert child == mRoot;
- notifyItemRangeChanged(itemPosition, itemCount);
+ notifyItemRangeChanged(itemPosition, itemCount, payload);
}
@Override

Powered by Google App Engine
This is Rietveld 408576698