[Contextual Search] Adds basic support for narrow Search Panel.

chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java

Index: chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java
index 231c50c3c21187dbd02b4e678ba562ec7e7b4b7b..894e82d66c54e05d4e905c62f8afb1fb1d5cb0ae 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/eventfilter/ContextualSearchEventFilter.java
@@ -181,7 +181,8 @@ public class ContextualSearchEventFilter extends GestureEventFilter {
 
         if (!mIsDeterminingEventTarget && action == MotionEvent.ACTION_DOWN) {
             mInitialEventY = e.getY();
-            if (mSearchPanel.isYCoordinateInsideSearchContentView(mInitialEventY * mPxToDp)) {
+            if (mSearchPanel.isYCoordinateInsideSearchContentView(
+                    e.getX() * mPxToDp, mInitialEventY * mPxToDp)) {
                 // If the DOWN event happened inside the Search Content View, we'll need
                 // to wait until the user has moved the finger beyond a certain threshold,
                 // so we can determine the gesture's orientation and consequently be able
@@ -357,10 +358,11 @@ public class ContextualSearchEventFilter extends GestureEventFilter {
             isSyntheticEvent = true;
         }
 
-        float searchContentViewOffsetYPx = mSearchPanel.getSearchContentViewOffsetY() / mPxToDp;
+        final float contentViewOffsetXPx = mSearchPanel.getOffsetX() / mPxToDp;

[donnd, 2015/07/14 22:24:13]

Should this called method be named getSearchContentViewOffsetX?

[David Trainor- moved to gerrit, 2015/07/14 23:12:00]

+1

[pedro (no code reviews), 2015/07/15 00:19:16]

Done.

[pedro (no code reviews), 2015/07/15 00:19:16]

Good point. Also added separate method for the width and height of the
ContentView.

+        final float contentViewOffsetYPx = mSearchPanel.getSearchContentViewOffsetY() / mPxToDp;
 
         // Adjust the offset to be relative to the Search Contents View.
-        event.offsetLocation(0.f, -searchContentViewOffsetYPx);
+        event.offsetLocation(-contentViewOffsetXPx, -contentViewOffsetYPx);
 
         boolean wasEventCanceled = false;
         if (mWasActionDownEventSynthetic && action == MotionEvent.ACTION_UP) {
@@ -404,7 +406,8 @@ public class ContextualSearchEventFilter extends GestureEventFilter {
      * @return Whether the event has been consumed.
      */
     private boolean handleSingleTapUp(MotionEvent e) {
-        setEventTarget(mSearchPanel.isYCoordinateInsideSearchContentView(e.getY() * mPxToDp)
+        setEventTarget(mSearchPanel.isYCoordinateInsideSearchContentView(
+                e.getX() * mPxToDp, e.getY() * mPxToDp)
                 ? EventTarget.SEARCH_CONTENT_VIEW : EventTarget.SEARCH_PANEL);
 
         return false;