Bug 23782 - ContainerNode::getLowerRightCorner() miscalculate for text node.
Summary: ContainerNode::getLowerRightCorner() miscalculate for text node.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: DOM (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-05 21:42 PST by Lyon Chen
Modified: 2009-02-13 13:50 PST (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lyon Chen 2009-02-05 21:42:49 PST
In function ContainerNode::getLowerRightCorner(), the line

point.move(linesBox.x() + linesBox.width(), linesBox.height());

should be

point.move(linesBox.x() + linesBox.width(), linesBox.y() + linesBox.height());

Otherwise for text node the right corner position is wrong.
Comment 1 Adam Treat 2009-02-13 06:53:59 PST
Lyon, can you come up with a layouttest/testcase for this?
Comment 2 Lyon Chen 2009-02-13 07:16:39 PST
I will come up with a test page. In WinMo phones, because of this error we can not move up focus from "sales@torchmobile.com" to "Torch Mobile Inc" with arrow keyboard in Iris Browser about page.
Comment 3 Adam Treat 2009-02-13 13:50:01 PST
Was reviewed by David Hyatt over IRC and landed by me r40991.