Chromium Code Reviews| Index: chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
| diff --git a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
| index 39b829925c41e0ed695718b050f7dcfc92cafccd..193843b0fa0a8d059a5038241409589e97fd4124 100644 |
| --- a/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
| +++ b/chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarPhone.java |
| @@ -38,7 +38,6 @@ import android.view.animation.LinearInterpolator; |
| import android.widget.FrameLayout; |
| import android.widget.ImageView; |
| import android.widget.TextView; |
| -import android.widget.Toast; |
| import org.chromium.base.ApiCompatibilityUtils; |
| import org.chromium.base.SysUtils; |
| @@ -400,28 +399,11 @@ public class ToolbarPhone extends ToolbarLayout |
| @Override |
| public boolean onLongClick(View v) { |
| - int stringResourceId = -1; |
| + int stringResourceId = 0; |
| if (v == mToggleTabStackButton) { |
| stringResourceId = R.string.open_tabs; |
| } |
| - if (stringResourceId != -1) { |
| - Context ctx = getContext(); |
| - // Display the tooltip for the view being long clicked. |
| - final int screenWidth = getResources().getDisplayMetrics().widthPixels; |
| - final int[] screenPos = new int[2]; |
| - v.getLocationOnScreen(screenPos); |
| - final int width = v.getWidth(); |
| - |
| - Toast toast = Toast.makeText( |
| - ctx, getResources().getString(stringResourceId), Toast.LENGTH_SHORT); |
| - toast.setGravity( |
| - Gravity.TOP | Gravity.END, |
| - screenWidth - screenPos[0] - width / 2, |
| - getHeight()); |
| - toast.show(); |
| - return true; |
| - } |
| - return false; |
| + return showLongPressToast(v, stringResourceId); |
|
Yusuf
2015/06/29 18:03:17
else {
return false;
} here as well
Ian Wen
2015/06/29 18:27:42
Done.
|
| } |
| @Override |