Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(815)

Unified Diff: Source/core/editing/MarkupAccumulator.cpp

Issue 1161463008: Refactoring: Have StyledMarkupAccumulator use MarkupFormatter (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/editing/MarkupAccumulator.cpp
diff --git a/Source/core/editing/MarkupAccumulator.cpp b/Source/core/editing/MarkupAccumulator.cpp
index 4b88861fdacbf546e91065e81456fe7a88b60540..702ed5a438c26db81ec3931106031faa8a0748fe 100644
--- a/Source/core/editing/MarkupAccumulator.cpp
+++ b/Source/core/editing/MarkupAccumulator.cpp
@@ -77,28 +77,11 @@ void MarkupAccumulator::appendStartMarkup(StringBuilder& result, Node& node, Nam
case Node::TEXT_NODE:
appendText(result, toText(node));
break;
- case Node::COMMENT_NODE:
- MarkupFormatter::appendComment(result, toComment(node).data());
- break;
- case Node::DOCUMENT_NODE:
- MarkupFormatter::appendXMLDeclaration(result, toDocument(node));
- break;
- case Node::DOCUMENT_FRAGMENT_NODE:
- break;
- case Node::DOCUMENT_TYPE_NODE:
- MarkupFormatter::appendDocumentType(result, toDocumentType(node));
- break;
- case Node::PROCESSING_INSTRUCTION_NODE:
- MarkupFormatter::appendProcessingInstruction(result, toProcessingInstruction(node).target(), toProcessingInstruction(node).data());
- break;
case Node::ELEMENT_NODE:
appendElement(result, toElement(node), namespaces);
break;
- case Node::CDATA_SECTION_NODE:
- MarkupFormatter::appendCDATASection(result, toCDATASection(node).data());
- break;
- case Node::ATTRIBUTE_NODE:
- ASSERT_NOT_REACHED();
+ default:
+ m_formatter.appendStartMarkup(result, node, namespaces);
break;
}
}
@@ -120,11 +103,6 @@ void MarkupAccumulator::appendEndMarkup(StringBuilder& result, const Element& el
m_formatter.appendEndMarkup(result, element);
}
-void MarkupAccumulator::concatenateMarkup(StringBuilder& result) const
-{
- result.append(m_markup);
-}
-
void MarkupAccumulator::appendCustomAttributes(StringBuilder&, const Element&, Namespaces*)
{
}

Powered by Google App Engine
This is Rietveld 408576698