Refactor HitTestResult to store the HitTestRequest

Source/core/page/DragController.cpp

Index: Source/core/page/DragController.cpp
diff --git a/Source/core/page/DragController.cpp b/Source/core/page/DragController.cpp
index 1c4e2e5d616a7caf17b5e6029227b096b38bd792..43549d39875cd454344d61def4de52155bb00495 100644
--- a/Source/core/page/DragController.cpp
+++ b/Source/core/page/DragController.cpp
@@ -314,8 +314,8 @@ static Element* elementUnderMouse(Document* documentUnderMouse, const IntPoint&
     LayoutPoint point = roundedLayoutPoint(FloatPoint(p.x() * zoomFactor, p.y() * zoomFactor));
 
     HitTestRequest request(HitTestRequest::ReadOnly | HitTestRequest::Active);
-    HitTestResult result(point);
-    documentUnderMouse->layoutView()->hitTest(request, result);
+    HitTestResult result(request, point);
+    documentUnderMouse->layoutView()->hitTest(result);
 
     Node* n = result.innerNode();
     while (n && !n->isElementNode())
@@ -541,7 +541,8 @@ bool DragController::canProcessDrag(DragData* dragData)
         return false;
 
     IntPoint point = m_page->deprecatedLocalMainFrame()->view()->rootFrameToContents(dragData->clientPosition());
-    HitTestResult result = HitTestResult(point);
+    HitTestRequest request(HitTestRequest::ReadOnly | HitTestRequest::Active);
+    HitTestResult result = HitTestResult(request, point);

[Rick Byers, 2015/03/31 00:51:00]

nit: looks like you don't need these initializers at all in this case.  Just
remove these two lines and update the call to hitTestResultAtPoint below to be
'HitTestResult result = m_page...'

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

Done.

     if (!m_page->deprecatedLocalMainFrame()->contentRenderer())
         return false;