text-transform causes incorrect text placement in svg

text-transform causes incorrect text placement in svg

Override originalText method in RenderSVGInlineText so
that RenderText::transformText uses it and gets the
text with SVG whitespace rules applied.

BUG=327606
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184655

[rwlbuis, 2014-10-29 17:39:29]

rob.buis@samsung.com changed reviewers:
+ ed@opera.com

[rwlbuis, 2014-10-29 17:39:29]

PTAL

[fs, 2014-10-30 12:31:38]

fs@opera.com changed reviewers:
+ fs@opera.com

[fs, 2014-10-30 12:31:38]

LGTM (a bit uneasy about hiding the whitespace-mangling, but it sort of makes
sense as originalText() so...)

https://codereview.chromium.org/688023002/diff/40001/Source/core/rendering/sv...
File Source/core/rendering/svg/RenderSVGInlineText.cpp (right):

https://codereview.chromium.org/688023002/diff/40001/Source/core/rendering/sv...
Source/core/rendering/svg/RenderSVGInlineText.cpp:250: return
applySVGWhitespaceRules(result, style() ? (style()->whiteSpace() == PRE) :
false);
Nit: "style() && style()->whiteSpace() == PRE" feels less convoluted. (Could
also be applied in styleDidChange, but we can leave that for now.)

[rwlbuis, 2014-10-30 14:25:04]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2014-10-30 14:25:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/688023002/60001

[commit-bot: I haz the power, 2014-10-30 15:40:14]

Committed patchset #4 (id:60001) as 184655