[Webkit-unassigned] [Bug 54929] REGRESSION: Text selection broken for text with line-height applied

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=54929


Justin Garcia <justin.garcia at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |justin.garcia at apple.com




--- Comment #11 from Justin Garcia <justin.garcia at apple.com>  2011-03-17 13:04:28 PST ---
-        IntPoint point(pointInLogicalContents.x(), closestBox->logicalTop());
+        IntPoint point(pointInLogicalContents.x(), closestBox->isText() ? closestBox->root()->selectionTop() : closestBox->logicalHeight());

Why not pass selectionTop() in all cases?

RenderReplaced::positionForPoint(const IntPoint& point)

assumes that anything above selectionTop() is above it also, for example. Not sure about the others...

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.