Changeset 86705 in webkit for trunk/Source/WebCore/rendering/RenderFrameSet.cpp

Timestamp:

May 17, 2011, 1:57:28 PM (14 years ago)

Author:

[email protected]

Message:

2011-05-17 Emil A Eklund <​[email protected]>

Reviewed by Eric Seidel.

Change nodeAtPoint to take IntPoint instead of int x, int y
​https://bugs.webkit.org/show_bug.cgi?id=60663

Change nodeAtPoint to take a single const IntPoint& instead of a pair of
ints for the location.

Covered by existing tests.

• platform/graphics/FloatPoint.h: (WebCore::flooredIntPoint):
• rendering/EllipsisBox.cpp: (WebCore::EllipsisBox::nodeAtPoint):
• rendering/EllipsisBox.h:
• rendering/InlineBox.cpp: (WebCore::InlineBox::nodeAtPoint):
• rendering/InlineBox.h:
• rendering/InlineFlowBox.cpp: (WebCore::InlineFlowBox::nodeAtPoint):
• rendering/InlineFlowBox.h:
• rendering/InlineTextBox.cpp: (WebCore::InlineTextBox::nodeAtPoint):
• rendering/InlineTextBox.h:
• rendering/RenderBlock.cpp: (WebCore::RenderBlock::nodeAtPoint): (WebCore::RenderBlock::hitTestContents):
• rendering/RenderBlock.h:
• rendering/RenderBox.cpp: (WebCore::RenderBox::nodeAtPoint):
• rendering/RenderBox.h:
• rendering/RenderFrameSet.cpp: (WebCore::RenderFrameSet::nodeAtPoint):
• rendering/RenderFrameSet.h:
• rendering/RenderImage.cpp: (WebCore::RenderImage::nodeAtPoint):
• rendering/RenderImage.h:
• rendering/RenderInline.cpp: (WebCore::RenderInline::nodeAtPoint):
• rendering/RenderInline.h:
• rendering/RenderLineBoxList.cpp: (WebCore::RenderLineBoxList::hitTest):
• rendering/RenderListBox.cpp: (WebCore::RenderListBox::nodeAtPoint):
• rendering/RenderListBox.h:
• rendering/RenderObject.cpp: (WebCore::RenderObject::hitTest): (WebCore::RenderObject::nodeAtPoint):
• rendering/RenderObject.h:
• rendering/RenderTable.cpp: (WebCore::RenderTable::nodeAtPoint):
• rendering/RenderTable.h:
• rendering/RenderTableRow.cpp: (WebCore::RenderTableRow::nodeAtPoint):
• rendering/RenderTableRow.h:
• rendering/RenderTableSection.cpp: (WebCore::RenderTableSection::nodeAtPoint):
• rendering/RenderTableSection.h:
• rendering/RenderText.h: (WebCore::RenderText::nodeAtPoint):
• rendering/RenderTextControlMultiLine.cpp: (WebCore::RenderTextControlMultiLine::nodeAtPoint):
• rendering/RenderTextControlMultiLine.h:
• rendering/RenderTextControlSingleLine.cpp: (WebCore::RenderTextControlSingleLine::nodeAtPoint):
• rendering/RenderTextControlSingleLine.h:
• rendering/RenderWidget.cpp: (WebCore::RenderWidget::nodeAtPoint):
• rendering/RenderWidget.h:
• rendering/RootInlineBox.cpp: (WebCore::RootInlineBox::nodeAtPoint):
• rendering/RootInlineBox.h:
• rendering/svg/RenderSVGForeignObject.cpp: (WebCore::RenderSVGForeignObject::nodeAtFloatPoint): (WebCore::RenderSVGForeignObject::nodeAtPoint):
• rendering/svg/RenderSVGForeignObject.h:
• rendering/svg/RenderSVGModelObject.cpp: (WebCore::RenderSVGModelObject::nodeAtPoint):
• rendering/svg/RenderSVGModelObject.h:
• rendering/svg/RenderSVGRoot.cpp: (WebCore::RenderSVGRoot::nodeAtPoint):
• rendering/svg/RenderSVGRoot.h:
• rendering/svg/RenderSVGText.cpp: (WebCore::RenderSVGText::nodeAtFloatPoint): (WebCore::RenderSVGText::nodeAtPoint):
• rendering/svg/RenderSVGText.h:

File:

• trunk/Source/WebCore/rendering/RenderFrameSet.cpp (modified) (1 diff)

trunk/Source/WebCore/rendering/RenderFrameSet.cpp

@@ -159 +159 @@
 
 bool RenderFrameSet::nodeAtPoint(const HitTestRequest& request, HitTestResult& result,
-    int x, int y, int tx, int ty, HitTestAction action)
+    const IntPoint& pointInContainer, int tx, int ty, HitTestAction action)
 {
     if (action != HitTestForeground)
         return false;
 
-    bool inside = RenderBox::nodeAtPoint(request, result, x, y, tx, ty, action)
+    bool inside = RenderBox::nodeAtPoint(request, result, pointInContainer, tx, ty, action)
         || m_isResizing;