Reduce hit test on ShowPress by moving event targeting to WebViewImpl

Source/core/inspector/InspectorOverlay.cpp

Index: Source/core/inspector/InspectorOverlay.cpp
diff --git a/Source/core/inspector/InspectorOverlay.cpp b/Source/core/inspector/InspectorOverlay.cpp
index 5492252f41d193a4d6013440f386a5a5554920e4..dd4123b5ea2e71b59d5bf8bdb7e396adf2214191 100644
--- a/Source/core/inspector/InspectorOverlay.cpp
+++ b/Source/core/inspector/InspectorOverlay.cpp
@@ -44,6 +44,7 @@
 #include "core/loader/FrameLoadRequest.h"
 #include "core/page/Chrome.h"
 #include "core/page/EventHandler.h"
+#include "core/page/EventWithHitTestResults.h"
 #include "core/page/Page.h"
 #include "core/rendering/RenderBox.h"
 #include "core/rendering/RenderBoxModelObject.h"
@@ -369,7 +370,9 @@ bool InspectorOverlay::handleGestureEvent(const PlatformGestureEvent& event)
     if (isEmpty())
         return false;
 
-    return toLocalFrame(overlayPage()->mainFrame())->eventHandler().handleGestureEvent(event);
+    GestureEventWithHitTestResults targetedEvent =
+        toLocalFrame(overlayPage()->mainFrame())->eventHandler().targetGestureEvent(event);

[Rick Byers, 2014/08/20 23:59:32]

Perhaps you should keep a trivial version of EventHandler::handleGestureEvent
which just does these two steps.  Then you can avoid changing all these
callsites, but WebViewImpl can still separate the two steps.

[Zeeshan Qureshi, 2014/08/21 04:59:25]

Done.

+    return toLocalFrame(overlayPage()->mainFrame())->eventHandler().handleGestureEvent(targetedEvent);
 }
 
 bool InspectorOverlay::handleMouseEvent(const PlatformMouseEvent& event)