Refactor HitTestResult to store the HitTestRequest

Source/core/frame/LocalFrame.cpp

Index: Source/core/frame/LocalFrame.cpp
diff --git a/Source/core/frame/LocalFrame.cpp b/Source/core/frame/LocalFrame.cpp
index e9744442717815102b2be1a51a6a88b80dff7ada..7b406cc0be4771f40ccb79ee5d6e4395033f1f12 100644
--- a/Source/core/frame/LocalFrame.cpp
+++ b/Source/core/frame/LocalFrame.cpp
@@ -678,7 +678,8 @@ Document* LocalFrame::documentAtPoint(const IntPoint& pointInRootFrame)
         return nullptr;
 
     IntPoint pt = view()->rootFrameToContents(pointInRootFrame);
-    HitTestResult result = HitTestResult(pt);
+    HitTestRequest request(HitTestRequest::ReadOnly | HitTestRequest::Active);
+    HitTestResult result = HitTestResult(request, pt);

[Rick Byers, 2015/03/31 00:50:59]

looks like you should be able to avoid creating these local variables at all
with a little cleanup.  Eg.

if (!contentRenderer())
  return nullptr;

HitTestResult result = eventHandler()...;

return result ...;

[ramya.v, 2015/03/31 09:41:36]

Done.

 
     if (contentRenderer())
         result = eventHandler().hitTestResultAtPoint(pt, HitTestRequest::ReadOnly | HitTestRequest::Active);