commit | 7517ac4a99d96ba7fc06862f8f84b00d427d0827 | [log] [tgz] |
---|---|---|
author | Fredrik Söderquist <fs@opera.com> | Wed Dec 13 16:44:57 2017 |
committer | Commit Bot <commit-bot@chromium.org> | Wed Dec 13 16:44:57 2017 |
tree | 520142811a1ed42d51954ee0e01d2574f16a6224 | |
parent | 90553fe6e4980a3db0ffbfd930a8a77208ed1c27 [diff] |
[PE] Fix text selection of <svg:text> subject to 'textLength' With 'textLength' and 'lengthAdjust' with the value 'spacingAndGlyphs', each SVGTextFragment is subject to an additional scale factor that stretches the glyphs along the x- or y-axis. This scale factor was not (any longer) taken into account by SVGInlineTextBox::OffsetForPositionInFragment, which resulted in the incorrect offset being calculcated. Apply the (inverse) length_adjust_scale from SVGTextFragment before querying the font for an offset. Remove the |include_partial_glyphs| argument from SVGInlineTextBox::OffsetForPositionInFragment since it's always true, and also use 'float' rather LayoutUnit for the position. Remove the dead |horizontal_glyph_stretch_| member variable from TextRun since it's no longer used (since the simple shaper path was removed.) Bug: 793075 Change-Id: Ia7d369d914f5afd87d198b7b21077e180fc8bc59 Reviewed-on: https://chromium-review.googlesource.com/824162 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#523787}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .