Refactoring: Separate MarkupAccumulator into MarkupFormatter and MarkupAccumulator

Refactoring: Separate MarkupAccumulator into MarkupFormatter and MarkupAccumulator

Now MarkupAccumulator has two functions; one is to format a DOM node to
a string and the other is to accumulate the result of formatting. This
CL separates MarkupAccumulator into MarkupFormatter and
MarkupAccumulator for respective functions. MarkupFormatter only formats
nodes to texts, and (new) MarkupAccumulator uses MarkupFormatter and
accumulates the result. Now most of MarkupAccumulator functions
delegates their work to MarkupFormatter just by calling the samely named
functions. I plan to refactor those classes further more by reducing
redundant function calls.

This patch is result of collaboration work with yosin@chromium.org
for selection of web components. We've already prepared the new
node-traversal usages at crrev.com/1134473004. StyledMarkupAccumulator
is now using MarkupAccumulator, and I plan to change this to use
MarkupFormatter.

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

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

[hajimehoshi, 2015-06-03 10:58:10]

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

[hajimehoshi, 2015-06-03 10:58:10]

PTAL

[tkent, 2015-06-04 00:06:39]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-06-04 00:06:39]

lgtm

[commit-bot: I haz the power, 2015-06-04 00:07:17]

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

[commit-bot: I haz the power, 2015-06-04 00:10:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-06-04 00:10:47]

Try jobs failed on following builders:
  mac_blink_compile_dbg on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_compile_dbg/bu...)
  mac_blink_rel on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/mac_blink_rel/builds/57586)

[hajimehoshi, 2015-06-04 01:42:09]

The CQ bit was checked by hajimehoshi@chromium.org

[hajimehoshi, 2015-06-04 01:42:09]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1148633010/#ps20001
(title: "rebase")

[commit-bot: I haz the power, 2015-06-04 01:42:26]

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

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

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