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

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

Issue 2623993007: 🏠 Extract the ContentSuggestionManager interface from NTP (Closed)
Patch Set: aaaand rebase again 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/Footer.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/Footer.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/Footer.java
index 9fca3e1b25a7555083e5f816e2209191d8d728ab..e2b2fb83b98791db5f3cac123464e139f302128e 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/Footer.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/Footer.java
@@ -11,7 +11,7 @@
import android.widget.TextView;
import org.chromium.chrome.R;
-import org.chromium.chrome.browser.ntp.NewTabPageView.NewTabPageManager;
+import org.chromium.chrome.browser.suggestions.SuggestionsNavigationDelegate;
import org.chromium.ui.text.NoUnderlineClickableSpan;
import org.chromium.ui.text.SpanApplier;
@@ -35,7 +35,7 @@ protected void onBindViewHolder(NewTabPageViewHolder holder) {
* The {@code ViewHolder} for the {@link Footer}.
*/
public static class ViewHolder extends NewTabPageViewHolder {
- public ViewHolder(ViewGroup root, final NewTabPageManager manager) {
+ public ViewHolder(ViewGroup root, final SuggestionsNavigationDelegate navigationDelegate) {
super(LayoutInflater.from(root.getContext())
.inflate(R.layout.new_tab_page_footer, root, false));
@@ -43,7 +43,7 @@ public ViewHolder(ViewGroup root, final NewTabPageManager manager) {
@Override
public void onClick(View view) {
// TODO(mvanouwerkerk): Ensure this can be activated when using TalkBack.
- manager.onLearnMoreClicked();
+ navigationDelegate.navigateToHelpPage();
}
};

Powered by Google App Engine
This is Rietveld 408576698