Fold and remove SVGTextChunk

Fold and remove SVGTextChunk

Being the keeper of only some flags and a float, this class no longer
provide much in terms of value. Folding it up into it's users,
SVGText{,Path}ChunkBuilder gives less, and more straight-forward code.
This can also be considered a first step towards untying 'textLength'
from the chunk structure.

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

[fs, 2015-06-04 10:23:07]

fs@opera.com changed reviewers:
+ fmalita@chromium.org, pdr@chromium.org

[pdr., 2015-06-04 20:22:02]

So much cleaner! LGTM.

https://codereview.chromium.org/1160613004/diff/20001/Source/core/layout/svg/...
File Source/core/layout/svg/SVGTextChunkBuilder.cpp (right):

https://codereview.chromium.org/1160613004/diff/20001/Source/core/layout/svg/...
Source/core/layout/svg/SVGTextChunkBuilder.cpp:45: ETextAnchor textAnchor =
style.svgStyle().textAnchor();
Nit: These may be a little cleaner as a switch statement on
style.svgStyle().textAnchor() (here, and above). Up to your judgement.

[fs, 2015-06-05 08:42:32]

https://codereview.chromium.org/1160613004/diff/20001/Source/core/layout/svg/...
File Source/core/layout/svg/SVGTextChunkBuilder.cpp (right):

https://codereview.chromium.org/1160613004/diff/20001/Source/core/layout/svg/...
Source/core/layout/svg/SVGTextChunkBuilder.cpp:45: ETextAnchor textAnchor =
style.svgStyle().textAnchor();
On 2015/06/04 20:22:02, pdr wrote:
> Nit: These may be a little cleaner as a switch statement on
> style.svgStyle().textAnchor() (here, and above). Up to your judgement.

I had this in PS1, but then some of the bots bombed on it... I suppose I could
be more persuasive (add default case.) Done (hopefully.)

[fs, 2015-06-05 10:21:28]

The CQ bit was checked by fs@opera.com

[fs, 2015-06-05 10:21:29]

The patchset sent to the CQ was uploaded after l-g-t-m from pdr@chromium.org
Link to the patchset: https://codereview.chromium.org/1160613004/#ps40001
(title: "Switch again.")

[commit-bot: I haz the power, 2015-06-05 10:21:49]

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

[commit-bot: I haz the power, 2015-06-05 10:25:15]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196567