Refactoring: Have StyledMarkupAccumulator use MarkupFormatter

Refactoring: Have StyledMarkupAccumulator use MarkupFormatter

StyledMarkupAccumulator is now using MarkupAccumulator. This CL replaces
this usage with MarkupFormatter. MarkupAccumulator has been separated
into (new) MarkupAccumulator and MarkupFormatter at crrev.com/1148633010.
MarkupFormatter focuses on formatting DOM nodes to texts, and it is
better for StyledMarkupAccumulator to use MarkupFormatter instead of
MarkupAccumulator for code health.

BUG=n/a
TEST=n/a; no behavior change

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

[hajimehoshi, 2015-06-04 05:41:33]

hajimehoshi@chromium.org changed reviewers:
+ tkent@chromium.org, yosin@chromium.org

[hajimehoshi, 2015-06-04 05:41:34]

PTAL

[yosin_UTC9, 2015-06-04 06:05:41]

https://codereview.chromium.org/1161463008/diff/20001/Source/core/editing/mar...
File Source/core/editing/markup.cpp (right):

https://codereview.chromium.org/1161463008/diff/20001/Source/core/editing/mar...
Source/core/editing/markup.cpp:49: #include "core/editing/MarkupAccumulator.h"
Q: Do we need to have this include?

[hajimehoshi, 2015-06-04 06:07:39]

https://codereview.chromium.org/1161463008/diff/20001/Source/core/editing/mar...
File Source/core/editing/markup.cpp (right):

https://codereview.chromium.org/1161463008/diff/20001/Source/core/editing/mar...
Source/core/editing/markup.cpp:49: #include "core/editing/MarkupAccumulator.h"
On 2015/06/04 06:05:41, Yosi_UTC9 wrote:
> Q: Do we need to have this include?

Yes. MarkupAccumulator is used in this file, and it seems like that
MarkupAccumulator was indirectly included by StyledMarkupSerializer. Now
StyledMarkupSerializer/Accumulator have nothing to do with MarkupAccumulator, we
need to add this #include.

[yosin_UTC9, 2015-06-04 06:09:42]

lgtm

[yosin_UTC9, 2015-06-04 06:09:46]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-06-04 06:09:53]

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

[commit-bot: I haz the power, 2015-06-04 07:34:16]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196474