diff options
author | Simon Hausmann <simon.hausmann@nokia.com> | 2012-07-30 11:37:48 +0200 |
---|---|---|
committer | Simon Hausmann <simon.hausmann@nokia.com> | 2012-07-30 11:38:52 +0200 |
commit | 89e2486a48b739f8d771d69ede5a6a1b244a10fc (patch) | |
tree | 503b1a7812cf97d93704c32437eb5f62dc1a1ff9 /Source/WebCore/rendering/RenderFlowThread.cpp | |
parent | 625f028249cb37c55bbbd153f3902afd0b0756d9 (diff) | |
download | qtwebkit-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.cpp | 13 |
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 |