Hit testing in table cells with top/bottom space from vertical alignment didn't...

	super-secret yPos() lie that table cells do.  Use m_y instead of yPos().

        * khtml/rendering/render_block.cpp:
        (khtml::RenderBlock::nodeAtPoint):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@8076 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebCore/ChangeLog-2005-08-23 b/WebCore/ChangeLog-2005-08-23
index bf055fca1e9cc7d1a443cd8873f196c1087a8cb0..0df78e0441526d60166d923d5d97ee80fc055b98 100644 (file)
--- a/WebCore/ChangeLog-2005-08-23
+++ b/WebCore/ChangeLog-2005-08-23
@@ -1,3 +1,11 @@
+2004-11-23  David Hyatt  <hyatt@apple.com>
+
+       Hit testing in table cells with top/bottom space from vertical alignment didn't work.  I forgot about the
+       super-secret yPos() lie that table cells do.  Use m_y instead of yPos().
+
+        * khtml/rendering/render_block.cpp:
+        (khtml::RenderBlock::nodeAtPoint):
+
 2004-11-22  David Hyatt  <hyatt@apple.com>
 
        Make sure you can use document.createElement to make a <canvas> element.

diff --git a/WebCore/khtml/rendering/render_block.cpp b/WebCore/khtml/rendering/render_block.cpp
index ccfbc39fd634b3dc99e472947f6cf28eba8b6994..1c024c000dcbab3803eef9845cf7ea0602240ffe 100644 (file)
--- a/WebCore/khtml/rendering/render_block.cpp
+++ b/WebCore/khtml/rendering/render_block.cpp
@@ -2337,8 +2337,8 @@ bool RenderBlock::nodeAtPoint(NodeInfo& info, int _x, int _y, int _tx, int _ty,
 {
     bool inlineFlow = isInlineFlow();
 
-    int tx = _tx + xPos();
-    int ty = _ty + yPos();
+    int tx = _tx + m_x;
+    int ty = _ty + m_y;
 
     if (!inlineFlow && !isRoot()) {
         // Check if we need to do anything at all with this block.