summaryrefslogtreecommitdiff
path: root/Source/WebCore/rendering/RenderFlowThread.cpp
diff options
context:
space:
mode:
authorSimon Hausmann <simon.hausmann@nokia.com>2012-07-30 11:37:48 +0200
committerSimon Hausmann <simon.hausmann@nokia.com>2012-07-30 11:38:52 +0200
commit89e2486a48b739f8d771d69ede5a6a1b244a10fc (patch)
tree503b1a7812cf97d93704c32437eb5f62dc1a1ff9 /Source/WebCore/rendering/RenderFlowThread.cpp
parent625f028249cb37c55bbbd153f3902afd0b0756d9 (diff)
downloadqtwebkit-89e2486a48b739f8d771d69ede5a6a1b244a10fc.tar.gz
Imported WebKit commit 0282df8ca7c11d8c8a66ea18543695c69f545a27 (http://svn.webkit.org/repository/webkit/trunk@124002)
New snapshot with prospective Mountain Lion build fix
Diffstat (limited to 'Source/WebCore/rendering/RenderFlowThread.cpp')
-rw-r--r--Source/WebCore/rendering/RenderFlowThread.cpp13
1 files changed, 4 insertions, 9 deletions
diff --git a/Source/WebCore/rendering/RenderFlowThread.cpp b/Source/WebCore/rendering/RenderFlowThread.cpp
index 669d46d4d..1b09cbb8d 100644
--- a/Source/WebCore/rendering/RenderFlowThread.cpp
+++ b/Source/WebCore/rendering/RenderFlowThread.cpp
@@ -315,18 +315,13 @@ bool RenderFlowThread::hitTestRegion(RenderRegion* region, const HitTestRequest&
} else
renderFlowThreadOffset = accumulatedOffset - regionRect.location();
- LayoutPoint transformedPoint = pointInContainer.point() - renderFlowThreadOffset;
-
// Always ignore clipping, since the RenderFlowThread has nothing to do with the bounds of the FrameView.
HitTestRequest newRequest(request.type() | HitTestRequest::IgnoreClipping);
- RenderRegion* oldRegion = result.region();
- result.setRegion(region);
- LayoutPoint oldPoint = result.point();
- result.setPoint(transformedPoint);
- bool isPointInsideFlowThread = layer()->hitTest(newRequest, result);
- result.setPoint(oldPoint);
- result.setRegion(oldRegion);
+ // Make a new temporary hitTestPoint in the new region.
+ HitTestPoint newHitTestPoint(pointInContainer, -renderFlowThreadOffset, region);
+
+ bool isPointInsideFlowThread = layer()->hitTest(newRequest, newHitTestPoint, result);
// FIXME: Should we set result.m_localPoint back to the RenderRegion's coordinate space or leave it in the RenderFlowThread's coordinate
// space? Right now it's staying in the RenderFlowThread's coordinate space, which may end up being ok. We will know more when we get around to