Move StyledMarkupAccumulator::shouldApplyWrappingStyle to its serializer

Source/core/editing/StyledMarkupAccumulator.h

Index: Source/core/editing/StyledMarkupAccumulator.h
diff --git a/Source/core/editing/StyledMarkupAccumulator.h b/Source/core/editing/StyledMarkupAccumulator.h
index d885cc6bc98614f3e0c579f0df7af10a0f7c232a..5ca821bfb2971c4e4e4043bed76b99395755d619 100644
--- a/Source/core/editing/StyledMarkupAccumulator.h
+++ b/Source/core/editing/StyledMarkupAccumulator.h
@@ -46,11 +46,10 @@ class StyledMarkupAccumulator final {
 public:
     StyledMarkupAccumulator(EAbsoluteURLs, const TextOffset& start, const TextOffset& end, const PassRefPtrWillBeRawPtr<Document>, EAnnotateForInterchange, Node*);
 
-    void appendStartTag(Node&);
     void appendEndTag(const Element&);
     void appendInterchangeNewline();
-
     void appendText(Text&);
+    void appendTextWithWrappingStyle(Text&);

[yosin_UTC9, 2015/06/12 12:09:55]

how about to call this function |appendTextWithInlineStyle|? Since, we have
|appendElementWithInlineStyle()| at L67.

[hajimehoshi, 2015/06/16 07:03:27]

Hmm, in the case of text nodes, a new <span> element is created to wrap the text
and the style is applied. That's the difference.

[hajimehoshi, 2015/06/16 08:15:18]

Done.

 
     // TODO(hajimehoshi): These functions are called from the serializer, but
     // should not.
@@ -71,8 +70,6 @@ public:
     bool shouldApplyWrappingStyle(const Node&) const;
 
 private:
-    void appendText(StringBuilder&, Text&);
-
     String renderedText(Text&);
     String stringValueForRange(const Text&);
     bool shouldAnnotate() const;