Refactor HitTestResult to store the HitTestRequest

Source/core/frame/LocalFrame.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Simon Hausmann <hausmann@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
@@ skipping 660 matching lines @@
         visiblePos = VisiblePosition(firstPositionInOrBeforeNode(node));
     return visiblePos;
 }
 
 Document* LocalFrame::documentAtPoint(const IntPoint& pointInRootFrame)
 {
     if (!view())
         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);
     return result.innerNode() ? &result.innerNode()->document() : nullptr;
 }
 
 PassRefPtrWillBeRawPtr<Range> LocalFrame::rangeForPoint(const IntPoint& framePoint)
 {
     VisiblePosition position = visiblePositionForPoint(framePoint);
     if (position.isNull())
@@ skipping 105 matching lines @@
     // We detach the FrameView's custom scroll bars as early as
     // possible to prevent m_doc->detach() from messing with the view
     // such that its scroll bars won't be torn down.
     //
     // FIXME: We should revisit this.
     if (m_view)
         m_view->prepareForDetach();
 }
 
 } // namespace blink