Refactoring: Remove StyledMarkupAccumulator::appendString and add appendNewLine

Source/core/editing/StyledMarkupAccumulator.cpp

 /*
  * 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 32 matching lines @@
     : m_formatter(shouldResolveURLs)
     , m_start(start)
     , m_end(end)
     , m_document(document)
     , m_shouldAnnotate(shouldAnnotate)
     , m_convertBlocksToInlines(convertBlocksToInlines)
     , m_highestNodeToBeSerialized(highestNodeToBeSerialized)
 {
 }
 
-void StyledMarkupAccumulator::appendString(const String& str)
-{
-    m_result.append(str);
-}
-
 void StyledMarkupAccumulator::appendStartTag(Node& node)
 {
     appendStartMarkup(m_result, node);
 }
 
 void StyledMarkupAccumulator::appendEndTag(const Element& element)
 {
     appendEndMarkup(m_result, element);
 }
 
@@ skipping 140 matching lines @@
     // wrappingStyleForSerialization should have removed -webkit-text-decorations-in-effect
     ASSERT(propertyMissingOrEqualToNone(style, CSSPropertyWebkitTextDecorationsInEffect));
     ASSERT(m_document);
 
     StringBuilder openTag;
     openTag.appendLiteral("<div style=\"");
     MarkupFormatter::appendAttributeValue(openTag, style->asText(), m_document->isHTMLDocument());
     openTag.appendLiteral("\">");
     m_reversedPrecedingMarkup.append(openTag.toString());
 
-    appendString("</div>");
+    m_result.append("</div>");
 }
 
 String StyledMarkupAccumulator::takeResults()
 {
     StringBuilder result;
     result.reserveCapacity(MarkupFormatter::totalLength(m_reversedPrecedingMarkup) + m_result.length());
 
     for (size_t i = m_reversedPrecedingMarkup.size(); i > 0; --i)
         result.append(m_reversedPrecedingMarkup[i - 1]);
 
@@ skipping 32 matching lines @@
     // TODO(hajimehoshi): Use Strategy
     return m_highestNodeToBeSerialized && m_highestNodeToBeSerialized->parentNode() == node.parentNode()
         && m_wrappingStyle && m_wrappingStyle->style();
 }
 
 bool StyledMarkupAccumulator::shouldAnnotate() const
 {
     return m_shouldAnnotate == AnnotateForInterchange;
 }
 
+void StyledMarkupAccumulator::appendNewLine()

[yosin_UTC9, 2015/06/10 07:06:33]

s/NewLine/newline/
as "interchange Newline String" in below.

[hajimehoshi, 2015/06/10 07:16:30]

Done.

+{
+    // TODO(hajimehoshi): This is copied from
+    // StyledMarkupSerialize::createMarkup. Unify them.
+    DEFINE_STATIC_LOCAL(const String, interchangeNewlineString, ("<br class=\"" AppleInterchangeNewline "\">"));
+    m_result.append(interchangeNewlineString);
+}
+
 } // namespace blink