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

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

Issue 1374533003: Remove NTP toasts that we no longer show. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java
index cea35b8f3a4537ff4028d324147cd9828601d854..59694ea7c66e7786cb44fa5645659f0ee5117e56 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageToolbar.java
@@ -6,27 +6,22 @@ package org.chromium.chrome.browser.ntp;
import android.content.Context;
import android.util.AttributeSet;
-import android.view.Gravity;
import android.view.View;
-import android.view.View.OnLongClickListener;
import android.view.ViewGroup;
import android.widget.LinearLayout;
import android.widget.TextView;
import org.chromium.base.ApiCompatibilityUtils;
import org.chromium.chrome.R;
-import org.chromium.chrome.browser.offlinepages.OfflinePageBridge;
import org.chromium.chrome.browser.widget.TintedDrawable;
-import org.chromium.ui.widget.Toast;
/**
* The toolbar at the bottom of the new tab page. Contains buttons to open the bookmarks and
* recent tabs pages.
*/
-public class NewTabPageToolbar extends LinearLayout implements OnLongClickListener {
+public class NewTabPageToolbar extends LinearLayout {
private View mBookmarksButton, mRecentTabsButton;
- private Toast mToast;
/**
* Constructor for inflating from xml.
@@ -59,28 +54,4 @@ public class NewTabPageToolbar extends LinearLayout implements OnLongClickListen
return button;
}
-
- @Override
- public boolean onLongClick(View v) {
- // Display tooltip on long click
- if (v == mBookmarksButton) {
- showTooltip(OfflinePageBridge.isEnabled()
- ? R.string.offline_pages_ntp_button_name : R.string.ntp_bookmarks);
- } else if (v == mRecentTabsButton) {
- showTooltip(R.string.recent_tabs);
- }
- return true;
- }
-
- /**
- * Shows a tooltip for a button. If a tooltip is already showing, it will be hidden.
- * @param stringId The string resource ID of the tooltip to be shown.
- */
- private void showTooltip(int stringId) {
- if (mToast != null) mToast.cancel();
- Context ctx = getContext();
- mToast = Toast.makeText(ctx, ctx.getResources().getString(stringId), Toast.LENGTH_SHORT);
- mToast.setGravity(Gravity.BOTTOM | Gravity.CENTER, 0, getHeight());
- mToast.show();
- }
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698