Refactoring: Remove the first parameter |result| from StyledMarkupAccumulator::appendStartMarkup

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 45 matching lines @@
     , m_start(start)
     , m_end(end)
     , m_document(document)
     , m_shouldAnnotate(shouldAnnotate)
     , m_highestNodeToBeSerialized(highestNodeToBeSerialized)
 {
 }
 
 void StyledMarkupAccumulator::appendStartTag(Node& node)
 {
-    appendStartMarkup(m_result, node);
+    appendStartMarkup(node);
 }
 
 void StyledMarkupAccumulator::appendEndTag(const Element& element)
 {
     appendEndMarkup(m_result, element);
 }
 
-void StyledMarkupAccumulator::appendStartMarkup(StringBuilder& result, Node& node)
+void StyledMarkupAccumulator::appendStartMarkup(Node& node)
 {
     switch (node.nodeType()) {
     case Node::TEXT_NODE:
-        appendText(toText(node));
+        appendText(m_result, toText(node));
         break;
     case Node::ELEMENT_NODE: {
         Element& element = toElement(node);
         RefPtrWillBeRawPtr<EditingStyle> style = createInlineStyle(element);
-        appendElement(element, style);
+        appendElementWithInlineStyle(m_result, element, style);

[yosin_UTC9, 2015/06/12 07:58:04]

Since |appendElement()| checks |element.isHTMLElement() && shouldAnnotate()) ||
shouldApplyWrappingStyle(element)|, is it OK to call
|appendElementWithInlineStyle()| without checking the condition?

         break;
     }
     default:
-        m_formatter.appendStartMarkup(result, node, nullptr);
+        m_formatter.appendStartMarkup(m_result, node, nullptr);
         break;
     }
 }
 
 void StyledMarkupAccumulator::appendEndMarkup(StringBuilder& result, const Element& element)
 {
     m_formatter.appendEndMarkup(result, element);
 }
 
-void StyledMarkupAccumulator::appendText(Text& text)
-{
-    appendText(m_result, text);
-}
-
 void StyledMarkupAccumulator::appendText(StringBuilder& out, Text& text)
 {
     const bool parentIsTextarea = text.parentElement() && text.parentElement()->tagQName() == textareaTag;
     const bool wrappingSpan = shouldApplyWrappingStyle(text) && !parentIsTextarea;
     if (wrappingSpan) {
         RefPtrWillBeRawPtr<EditingStyle> wrappingStyle = m_wrappingStyle->copy();
         // FIXME: <rdar://problem/5371536> Style rules that match pasted content can change it's appearance
         // Make sure spans are inline style in paste side e.g. span { display: block }.
         wrappingStyle->forceInline();
         // FIXME: Should this be included in forceInline?
@@ skipping 163 matching lines @@
     m_reversedPrecedingMarkup.append(str);
 }
 
 void StyledMarkupAccumulator::appendInterchangeNewline()
 {
     DEFINE_STATIC_LOCAL(const String, interchangeNewlineString, ("<br class=\"" AppleInterchangeNewline "\">"));
     m_result.append(interchangeNewlineString);
 }
 
 } // namespace blink