Move StyledMarkupAccumulator::shouldApplyWrappingStyle to its serializer

Source/core/editing/StyledMarkupAccumulator.h

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009, 2010, 2011 Google Inc. All rights reserved.
  * Copyright (C) 2011 Igalia S.L.
  * Copyright (C) 2011 Motorola Mobility. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
@@ skipping 28 matching lines @@
 class Document;
 class StylePropertySet;
 class Text;
 
 class StyledMarkupAccumulator final {
     WTF_MAKE_NONCOPYABLE(StyledMarkupAccumulator);
     STACK_ALLOCATED();
 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.
     Node* highestNodeToBeSerialized() { return m_highestNodeToBeSerialized.get(); }
     void setHighestNodeToBeSerialized(Node* highestNodeToBeSerialized) { m_highestNodeToBeSerialized = highestNodeToBeSerialized; }
     void setWrappingStyle(PassRefPtrWillBeRawPtr<EditingStyle> wrappingStyle) { m_wrappingStyle = wrappingStyle; }
 
     void wrapWithStyleNode(StylePropertySet*);
     String takeResults();
 
     void pushMarkup(const String&);
     RefPtrWillBeRawPtr<EditingStyle> createInlineStyle(Element&);
 
     void appendElement(StringBuilder&, const Element&);
     void appendElementWithInlineStyle(StringBuilder&, const Element&, PassRefPtrWillBeRawPtr<EditingStyle>);
     void appendStartMarkup(Node&);
 
     bool shouldApplyWrappingStyle(const Node&) const;
 
 private:
-    void appendText(StringBuilder&, Text&);
-
     String renderedText(Text&);
     String stringValueForRange(const Text&);
     bool shouldAnnotate() const;
 
     void appendElement(const Element&, PassRefPtrWillBeRawPtr<EditingStyle>);
     void appendEndMarkup(StringBuilder&, const Element&);
 
     MarkupFormatter m_formatter;
     const TextOffset m_start;
     const TextOffset m_end;
     const RefPtrWillBeMember<Document> m_document;
     const EAnnotateForInterchange m_shouldAnnotate;
     RawPtrWillBeMember<Node> m_highestNodeToBeSerialized;
     RefPtrWillBeMember<EditingStyle> m_wrappingStyle;
     StringBuilder m_result;
     Vector<String> m_reversedPrecedingMarkup;
 };
 
 } // namespace blink
 
 #endif // StyledMarkupAccumulator_h