Refactoring: Move StyledMarkupAccumulator::createInlineStyle to the 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&);
 
     // 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; }
+    PassRefPtrWillBeRawPtr<EditingStyle> wrappingStyle() const { return m_wrappingStyle; }

[tkent, 2015/06/16 22:27:00]

The return type should be |EditingStyle*|, or should change this function to
|copyWrappingStyle()| which returns a copy of m_wrappingStyle.
Without it, this change adds some extra instructions.

[hajimehoshi, 2015/06/17 04:33:51]

Done.

     void setWrappingStyle(PassRefPtrWillBeRawPtr<EditingStyle> wrappingStyle) { m_wrappingStyle = wrappingStyle; }
 
     void wrapWithStyleNode(StylePropertySet*);
     String takeResults();
 
     void pushMarkup(const String&);
-    RefPtrWillBeRawPtr<EditingStyle> createInlineStyle(Element&);
 
+    void appendElement(const Element&, PassRefPtrWillBeRawPtr<EditingStyle>);
     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&);
     void appendTextWithInlineStyle(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