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

Unified Diff: Source/core/dom/Node.cpp

Issue 14990005: Remove ENTITY_REFERENCE_NODE (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove some empty lines Created 7 years, 7 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
« no previous file with comments | « Source/core/dom/Node.h ('k') | Source/core/dom/Node.idl » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/Node.cpp
diff --git a/Source/core/dom/Node.cpp b/Source/core/dom/Node.cpp
index 1e80af0f00a8650f570082ee40da17996e303b16..aaf9fed13ba166a456faf26a217de30d3b1fb5a4 100644
--- a/Source/core/dom/Node.cpp
+++ b/Source/core/dom/Node.cpp
@@ -150,7 +150,6 @@ void Node::dumpStatistics()
size_t textNodes = 0;
size_t cdataNodes = 0;
size_t commentNodes = 0;
- size_t entityReferenceNodes = 0;
size_t entityNodes = 0;
size_t piNodes = 0;
size_t documentNodes = 0;
@@ -217,10 +216,6 @@ void Node::dumpStatistics()
++commentNodes;
break;
}
- case ENTITY_REFERENCE_NODE: {
- ++entityReferenceNodes;
- break;
- }
case ENTITY_NODE: {
++entityNodes;
break;
@@ -264,7 +259,6 @@ void Node::dumpStatistics()
printf(" Number of Text nodes: %zu\n", textNodes);
printf(" Number of CDATASection nodes: %zu\n", cdataNodes);
printf(" Number of Comment nodes: %zu\n", commentNodes);
- printf(" Number of EntityReference nodes: %zu\n", entityReferenceNodes);
printf(" Number of Entity nodes: %zu\n", entityNodes);
printf(" Number of ProcessingInstruction nodes: %zu\n", piNodes);
printf(" Number of Document nodes: %zu\n", documentNodes);
@@ -496,12 +490,6 @@ String Node::nodeValue() const
void Node::setNodeValue(const String& /*nodeValue*/, ExceptionCode& ec)
{
- // NO_MODIFICATION_ALLOWED_ERR: Raised when the node is readonly
- if (isReadOnlyNode()) {
- ec = NO_MODIFICATION_ALLOWED_ERR;
- return;
- }
-
// By default, setting nodeValue has no effect.
}
@@ -1027,11 +1015,6 @@ void Node::checkSetPrefix(const AtomicString& prefix, ExceptionCode& ec)
return;
}
- if (isReadOnlyNode()) {
- ec = NO_MODIFICATION_ALLOWED_ERR;
- return;
- }
-
// FIXME: Raise NAMESPACE_ERR if prefix is malformed per the Namespaces in XML specification.
const AtomicString& nodeNamespaceURI = namespaceURI();
@@ -1688,7 +1671,6 @@ static void appendTextContent(const Node* node, bool convertBRsToNewlines, bool&
// Fall through.
case Node::ATTRIBUTE_NODE:
case Node::ENTITY_NODE:
- case Node::ENTITY_REFERENCE_NODE:
case Node::DOCUMENT_FRAGMENT_NODE:
isNullString = false;
for (Node* child = node->firstChild(); child; child = child->nextSibling()) {
@@ -1726,7 +1708,6 @@ void Node::setTextContent(const String& text, ExceptionCode& ec)
case ELEMENT_NODE:
case ATTRIBUTE_NODE:
case ENTITY_NODE:
- case ENTITY_REFERENCE_NODE:
case DOCUMENT_FRAGMENT_NODE: {
RefPtr<ContainerNode> container = toContainerNode(this);
ChildListMutationScope mutation(this);
« no previous file with comments | « Source/core/dom/Node.h ('k') | Source/core/dom/Node.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698