Bug 40673 - offsetForPositionForComplexText() truncates float -> int on several platforms
Summary: offsetForPositionForComplexText() truncates float -> int on several platforms
Status: RESOLVED DUPLICATE of bug 92376
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 40665
Blocks:
  Show dependency tree / graph
 
Reported: 2010-06-16 05:16 PDT by Nikolas Zimmermann
Modified: 2013-01-28 02:21 PST (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolas Zimmermann 2010-06-16 05:16:34 PDT 
When bug 40665 lands, offsetForPositionForComplexText() no longer takes an 'int' parameter for the position, but a float. This is only relevant for SVG text, which supports subpixel positioning, and only for complex text.
Qt and Mac handle the float parameter properly, Win/Gtk etc not. That should be fixed, but has low importance.
Comment 1 Dominik Röttsches (drott) 2013-01-28 02:20:25 PST 
Nikolas, I'll close this as duplicate. For GTK & EFL this is fixed in bug 92376, landed as r124111. HarfBuzzShaper::offsetForPosition now takes a float argument and does the computation with this value without breaking it down or rounding it.

Please reopen if you think this is not completely fixed.

*** This bug has been marked as a duplicate of bug 92376 ***
Comment 2 Dominik Röttsches (drott) 2013-01-28 02:21:15 PST 
(In reply to comment #1)
> For GTK & EFL...

What i meant: For Harfbuzz-NG ports.