Reduce hit test on ShowPress by moving event targeting to WebViewImpl

Source/core/inspector/InspectorOverlay.cpp

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 26 matching lines @@
 #include "core/dom/PseudoElement.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
 #include "core/inspector/InspectorClient.h"
 #include "core/inspector/InspectorOverlayHost.h"
 #include "core/loader/EmptyClients.h"
 #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"
 #include "core/rendering/RenderInline.h"
 #include "core/rendering/RenderObject.h"
 #include "core/rendering/shapes/ShapeOutsideInfo.h"
 #include "core/rendering/style/RenderStyleConstants.h"
 #include "platform/JSONValues.h"
 #include "platform/PlatformMouseEvent.h"
 #include "platform/ScriptForbiddenScope.h"
@@ skipping 305 matching lines @@
 void InspectorOverlay::invalidate()
 {
     m_client->highlight();
 }
 
 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)
 {
     if (isEmpty())
         return false;
 
     EventHandler& eventHandler = toLocalFrame(overlayPage()->mainFrame())->eventHandler();
     switch (event.type()) {
     case PlatformEvent::MouseMoved:
@@ skipping 463 matching lines @@
     hideHighlight();
 }
 
 void InspectorOverlay::startedRecordingProfile()
 {
     if (!m_activeProfilerCount++)
         freePage();
 }
 
 } // namespace blink