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

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

Issue 2513453004: [Android NTP] Move suggestion sections into a separate node. (Closed)
Patch Set: sync 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: chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/AllDismissedItem.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/AllDismissedItem.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/AllDismissedItem.java
index 08404c3fb95a8ded578aac116ca63e405fe955d6..e03acc53a16f4b9f905556da5b7dfc90dba7e6c0 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/AllDismissedItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/AllDismissedItem.java
@@ -14,7 +14,6 @@ import android.widget.TextView;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.ntp.NewTabPageUma;
-import org.chromium.chrome.browser.ntp.NewTabPageView.NewTabPageManager;
import java.util.Calendar;
@@ -48,8 +47,7 @@ public class AllDismissedItem extends OptionalLeaf {
public static class ViewHolder extends NewTabPageViewHolder {
private final TextView mBodyTextView;
- public ViewHolder(
- ViewGroup root, final NewTabPageManager manager, final NewTabPageAdapter adapter) {
+ public ViewHolder(ViewGroup root, final SectionList sections) {
super(LayoutInflater.from(root.getContext())
.inflate(R.layout.new_tab_page_all_dismissed, root, false));
mBodyTextView = (TextView) itemView.findViewById(R.id.body_text);
@@ -60,9 +58,7 @@ public class AllDismissedItem extends OptionalLeaf {
public void onClick(View v) {
NewTabPageUma.recordAction(
NewTabPageUma.ACTION_CLICKED_ALL_DISMISSED_REFRESH);
- manager.getSuggestionsSource().restoreDismissedCategories();
- adapter.resetSections(/*allowEmptySections=*/true);
- manager.getSuggestionsSource().fetchRemoteSuggestions();
+ sections.restoreDismissedSections();
}
});
}

Powered by Google App Engine
This is Rietveld 408576698