Paste PopupMenu gets cropped by StatusBar

content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java

Index: content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java b/content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java
index 0745d207edb7da317513c811b2d5eafad3224367..cd53cae3cfea4699d90699b9cb4b46714e257a97 100644
--- a/content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java
+++ b/content/public/android/java/src/org/chromium/content/browser/input/PastePopupMenu.java
@@ -27,6 +27,7 @@ public class PastePopupMenu implements OnClickListener {
     private int mRawPositionY;
     private int mPositionX;
     private int mPositionY;
+    private int mStatusBarHeight;
     private final View[] mPasteViews;
     private final int[] mPasteViewLayouts;
     private final int mLineOffsetY;
@@ -81,6 +82,13 @@ public class PastePopupMenu implements OnClickListener {
                 5.0f, mContext.getResources().getDisplayMetrics());
         mWidthOffsetX = (int) TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP,
                 30.0f, mContext.getResources().getDisplayMetrics());
+
+        int statusBarHeightResourceId = mContext.getResources()

[jdduke (slow), 2014/07/31 17:07:39]

Prefer line breaking on arguments, or if the RHS can fit on an entire line,
after the equality, so these lines become:

final int statusBarHeightResourceId =
        mContext.getResources().getIdentifier("status_bar_height", "dimen",
"android");

and

mStatusBarHeight =
mContext.getResources().getDimensionPixelSize(statusBarHeightResourceId);

[kingshuk.j, 2014/08/01 07:55:44]

Done.

+                .getIdentifier("status_bar_height", "dimen", "android");
+        if (statusBarHeightResourceId > 0) {
+            mStatusBarHeight = mContext.getResources()
+                    .getDimensionPixelSize(statusBarHeightResourceId);
+        }
     }
 
     /**
@@ -131,8 +139,13 @@ public class PastePopupMenu implements OnClickListener {
         coords[0] += mPositionX;
         coords[1] += mPositionY;
 
+        int minOffsetY = 0;
+        if (mParent.getSystemUiVisibility() == View.SYSTEM_UI_FLAG_VISIBLE) {
+            minOffsetY = mStatusBarHeight;
+        }
+
         final int screenWidth = mContext.getResources().getDisplayMetrics().widthPixels;
-        if (coords[1] < 0) {
+        if (coords[1] < minOffsetY) {
             updateContent(false);
             // Update dimensions from new view
             contentView = mContainer.getContentView();