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

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

Issue 2122993003: Remove unused Interests code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Forgot unit test target. Created 4 years, 5 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/NewTabPageView.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
index cb60db70713415b2311dda7a3a95b44de7c1db7c..09d43320601b74d45e45811ccac08ce51c0f440c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
@@ -128,9 +128,6 @@ public class NewTabPageView extends FrameLayout
/** @return Whether voice search is enabled and the microphone should be shown. */
boolean isVoiceSearchEnabled();
- /** @return Whether the NTP Interests tab is enabled and its button should be shown. */
- boolean isInterestsEnabled();
-
/** @return Whether the toolbar at the bottom of the NTP is enabled and should be shown. */
boolean isToolbarEnabled();
@@ -152,9 +149,6 @@ public class NewTabPageView extends FrameLayout
/** Opens a url in the current tab. */
void openUrl(String url);
- /** Opens the interests dialog. */
- void navigateToInterests();
-
/**
* Animates the search box up into the omnibox and bring up the keyboard.
* @param beginVoiceSearch Whether to begin a voice search.
@@ -351,17 +345,6 @@ public class NewTabPageView extends FrameLayout
mManager.navigateToBookmarks();
}
});
- toolbar.getInterestsButton().setOnClickListener(new View.OnClickListener() {
- @Override
- public void onClick(View v) {
- mManager.navigateToInterests();
- }
- });
-
- // Set up interests
- if (manager.isInterestsEnabled()) {
- toolbar.getInterestsButton().setVisibility(View.VISIBLE);
- }
} else {
((ViewGroup) toolbar.getParent()).removeView(toolbar);
MarginLayoutParams params = (MarginLayoutParams) getWrapperView().getLayoutParams();

Powered by Google App Engine
This is Rietveld 408576698