Move painting code from SVGInlineTextBox to SVGInlineTextBoxPainter.

Move painting code from SVGInlineTextBox to SVGInlineTextBoxPainter.

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

[chrishtr, 2014-10-03 19:08:07]

chrishtr@chromium.org changed reviewers:
+ pdr@chromium.org

[pdr., 2014-10-03 21:44:16]

LGTM, rs=me. There are compile failures, I think you are just missing a forward
declaration.

[fs, 2014-10-06 08:34:52]

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

[fs, 2014-10-06 08:34:53]

On 2014/10/03 21:44:16, pdr wrote:
> LGTM, rs=me. There are compile failures, I think you are just missing a
forward
> declaration.

More likely missing some "m_svgInlineTextBox." prefixes. Also, not reordering
the functions while moving them might have given a nicer (read: more obvious)
diff.

And now to prepare for the conflicts this will inflict on me when it lands...
;-)

https://codereview.chromium.org/627773002/diff/60001/Source/core/paint/SVGInl...
File Source/core/paint/SVGInlineTextBoxPainter.cpp (right):

https://codereview.chromium.org/627773002/diff/60001/Source/core/paint/SVGInl...
Source/core/paint/SVGInlineTextBoxPainter.cpp:83: if (textRenderer.frame() &&
textRenderer.frame()->view() && textRenderer.frame()->view()->paintBehavior() &
PaintBehaviorRenderingSVGMask) {
You might hit a conflict here (https://codereview.chromium.org/625933003/ +
follow up)

https://codereview.chromium.org/627773002/diff/60001/Source/core/paint/SVGInl...
Source/core/paint/SVGInlineTextBoxPainter.cpp:289: ASSERT(resourceModeFlags !=
ApplyToDefaultMode);
...and here (https://codereview.chromium.org/622393002/). One more instance
below.

[chrishtr, 2014-10-06 23:15:06]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2014-10-06 23:15:41]

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

[commit-bot: I haz the power, 2014-10-07 02:08:28]

Committed patchset #6 (id:100001) as 183298